[ARROW-8537] [C++] Performance regression from ARROW-8523

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.0.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/24705
• Assignee: Yibo Cai
• Reporter: Yibo Cai
• Created: 21/Apr/20 02:55
• Updated: 11/Jan/23 08:01
• Resolved: 22/Apr/20 12:04

Description

I optimized BitmapReader in this PR and see performance uplift of BitmapReader test case. I didn't check other test cases as this change looks trivial.
I reviewed all test cases just now and see big performance drop of 4 cases, details at PR link.
I also compared performance of code using BitmapReader, no obvious changes found. Looks we should revert that PR.