[Optimize](parquet-reader) Add has_filter template param in get_next_run() to decrease _has_filter condition checking count in the loop.

[kaka11chen]

Proposed changes

Problem summary

In XXXDecoder::decode_values():

while (size_t run_length = select_vector.get_next_run<has_filter>(&read_type)) {
 ... 
}

It will call get_next_run(), need to check _has_filter in the loop.

size_t get_next_run(DataReadType* data_read_type) {
    if (_has_filter) {
    
    } else {
  
    }
}

It is unnecessary. So optimize it by adding has_filter template param in get_next_run() to decrease _has_filter condition checking count in the loop.

Checklist(Required)

• Does it affect the original behavior
• Has unit tests been added
• Has document been added or modified
• Does it need to update dependencies
• Is this PR support rollback (If NO, please explain WHY)

Further comments

If this is a relatively large or complex change, kick off the discussion at [email protected] by explaining why you chose the solution you did and what alternatives you considered, etc...

[github-actions]

clang-tidy review says "All clean, LGTM! 👍"

[kaka11chen]

run buildall

[hello-stephen]

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 33.92 seconds
stream load tsv: 429 seconds loaded 74807831229 Bytes, about 166 MB/s
stream load json: 24 seconds loaded 2358488459 Bytes, about 93 MB/s
stream load orc: 58 seconds loaded 1101869774 Bytes, about 18 MB/s
stream load parquet: 31 seconds loaded 861443392 Bytes, about 26 MB/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230425081803_clickbench_pr_134684.html

[morningman]

LGTM

[github-actions]

PR approved by at least one committer and no changes requested.

[github-actions]

PR approved by anyone and no changes requested.

[AshinGau]

LGTM

[morningman]

LGTM