-
Notifications
You must be signed in to change notification settings - Fork 3.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Fix](Column) refactor ColumnNullable to provide flags safety #40769
Conversation
Co-authored-by: Jerry Hu <[email protected]>
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
run buildall |
TeamCity be ut coverage result: |
TPC-H: Total hot run time: 43231 ms
|
run buildall |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
TPC-H: Total hot run time: 43015 ms
|
TeamCity be ut coverage result: |
TPC-H: Total hot run time: 43007 ms
|
TPC-DS: Total hot run time: 195658 ms
|
run buildall |
TeamCity be ut coverage result: |
TPC-H: Total hot run time: 43321 ms
|
TPC-DS: Total hot run time: 195773 ms
|
ClickBench: Total hot run time: 31.08 s
|
…#40769) Issue Number: close #xxx In this pr we move null_map and relative flags to an individual base class to constrain its visibility. please reed the comment carefully test in master: ``` xxx/column_nullable_test.cpp:187: Failure Value of: null_dst->has_null() Actual: false Expected: true Error occurred in case0 [ FAILED ] ColumnNullableTest.PredicateTest (0 ms) [ RUN ] ColumnNullableTest.HashTest [ OK ] ColumnNullableTest.HashTest (0 ms) [----------] 3 tests from ColumnNullableTest (0 ms total) [----------] Global test environment tear-down [==========] 3 tests from 1 test suite ran. (1 ms total) [ PASSED ] 2 tests. [ FAILED ] 1 test, listed below: [ FAILED ] ColumnNullableTest.PredicateTest ``` and success after this pr --------- Co-authored-by: Jerry Hu <[email protected]>
…#40769) Issue Number: close #xxx In this pr we move null_map and relative flags to an individual base class to constrain its visibility. please reed the comment carefully test in master: ``` xxx/column_nullable_test.cpp:187: Failure Value of: null_dst->has_null() Actual: false Expected: true Error occurred in case0 [ FAILED ] ColumnNullableTest.PredicateTest (0 ms) [ RUN ] ColumnNullableTest.HashTest [ OK ] ColumnNullableTest.HashTest (0 ms) [----------] 3 tests from ColumnNullableTest (0 ms total) [----------] Global test environment tear-down [==========] 3 tests from 1 test suite ran. (1 ms total) [ PASSED ] 2 tests. [ FAILED ] 1 test, listed below: [ FAILED ] ColumnNullableTest.PredicateTest ``` and success after this pr --------- Co-authored-by: Jerry Hu <[email protected]>
…40769) (#40848) pick #40769 Co-authored-by: Jerry Hu <[email protected]>
## Proposed changes Issue Number: close #xxx In this pr we move null_map and relative flags to an individual base class to constrain its visibility. please reed the comment carefully test in master: ``` xxx/column_nullable_test.cpp:187: Failure Value of: null_dst->has_null() Actual: false Expected: true Error occurred in case0 [ FAILED ] ColumnNullableTest.PredicateTest (0 ms) [ RUN ] ColumnNullableTest.HashTest [ OK ] ColumnNullableTest.HashTest (0 ms) [----------] 3 tests from ColumnNullableTest (0 ms total) [----------] Global test environment tear-down [==========] 3 tests from 1 test suite ran. (1 ms total) [ PASSED ] 2 tests. [ FAILED ] 1 test, listed below: [ FAILED ] ColumnNullableTest.PredicateTest ``` and success after this pr --------- Co-authored-by: Jerry Hu <[email protected]>
Proposed changes
Issue Number: close #xxx
In this pr we move null_map and relative flags to an individual base class to constrain its visibility.
please reed the comment carefully
test in master:
and success after this pr