[Configurations](multi-catalog) Add enable_parquet_filter_by_min_max and enable_orc_filter_by_min_max Session variables.
#35012
Conversation
|
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
|
run buildall |
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| @@ -169,6 +170,7 @@ OrcReader::OrcReader(const TFileScanRangeParams& params, const TFileRangeDesc& r | |||
| _file_system(nullptr), | |||
| _io_ctx(io_ctx), | |||
| _enable_lazy_mat(enable_lazy_mat), | |||
| _enable_filter_by_min_max(true), | |||
There was a problem hiding this comment.
warning: member initializer for '_enable_filter_by_min_max' is redundant [modernize-use-default-member-init]
| _enable_filter_by_min_max(true), | |
| , |
kaka11chen
force-pushed
the
add_merge_filter_by_min_max_session_vars
branch
from
e792124
to
99324c6
Compare
|
run buildall |
kaka11chen
force-pushed
the
add_merge_filter_by_min_max_session_vars
branch
from
99324c6
to
6a3bb5b
Compare
|
run buildall |
kaka11chen
force-pushed
the
add_merge_filter_by_min_max_session_vars
branch
2 times, most recently
from
ac52f65
to
2d0ac02
Compare
| @@ -149,6 +149,8 @@ OrcReader::OrcReader(RuntimeProfile* profile, RuntimeState* state, | |||
| _ctz(ctz), | |||
| _io_ctx(io_ctx), | |||
| _enable_lazy_mat(enable_lazy_mat), | |||
| _enable_filter_by_min_max( | |||
| state == nullptr ? true : state->query_options().enable_orc_filter_by_min_max), | |||
There was a problem hiding this comment.
warning: member initializer for '_dict_cols_has_converted' is redundant [modernize-use-default-member-init]
| state == nullptr ? true : state->query_options().enable_orc_filter_by_min_max), | |
| , |
| @@ -169,6 +171,7 @@ | |||
| _file_system(nullptr), | |||
| _io_ctx(io_ctx), | |||
| _enable_lazy_mat(enable_lazy_mat), | |||
| _enable_filter_by_min_max(true), | |||
There was a problem hiding this comment.
warning: member initializer for '_dict_cols_has_converted' is redundant [modernize-use-default-member-init]
| _enable_filter_by_min_max(true), | |
| { |
…` and `enable_orc_filter_by_min_max` Session variables.
kaka11chen
force-pushed
the
add_merge_filter_by_min_max_session_vars
branch
from
2d0ac02
to
5aa608d
Compare
|
run buildall |
|
TeamCity be ut coverage result: |
github-actions
bot
added
the
approved
|
PR approved by at least one committer and no changes requested. |
|
PR approved by anyone and no changes requested. |
…` and `enable_orc_filter_by_min_max` Session variables. (apache#35012)
…` and `enable_orc_filter_by_min_max` Session variables. (apache#35012)
…` and `enable_orc_filter_by_min_max` Session variables. (apache#35012)
…` and `enable_orc_filter_by_min_max` Session variables. (apache#35012)
…` and `enable_orc_filter_by_min_max` Session variables. (apache#35012)
…ionality, the mistake for apache#35012.
…` and `enable_orc_filter_by_min_max` Session variables. (apache#35012)
…` and `enable_orc_filter_by_min_max` Session variables. (apache#35012)
…` and `enable_orc_filter_by_min_max` Session variables. (#35012)
…ionality, the mistake for apache#35012. (apache#35320) fix bug introduced from apache#35012
Proposed changes
[Configurations] (multi-catalog) Add
enable_parquet_filter_by_min_maxandenable_orc_filter_by_min_maxSession variables for workaround.In case that when the column min max statistics in file are incorrect, we can set these variables to skip them
Further comments
If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...