eldenmoon commented on PR #47688: URL: https://github.com/apache/doris/pull/47688#issuecomment-2647140848
This pull request includes several changes to support inverted indexes for arrays and improve the handling of complex data types in the codebase. The most important changes include modifications to the inverted index writer, updates to expression evaluation, and the addition of new regression tests. ### Improvements to inverted index support: * [`be/src/olap/rowset/segment_v2/inverted_index_writer.cpp`](diffhunk://#diff-97781916b276f771710ab520c79ca29d5e4e331296fad7573fc9933a376dc165L79-R91): Updated `check_support_inverted_index` method to support one-level arrays and removed the check for extracted columns. * [`be/src/vec/exprs/vexpr.cpp`](diffhunk://#diff-46b3e9eb194dc83ddbdb52e26fc18f9204669761f0dc6f90502dc2b83828cb77L656-R656): Modified `_evaluate_inverted_index` method to remove nullable types from the target type during evaluation. ### Addition of regression tests: * [`regression-test/data/variant_p0/with_index/test_array_inverted_index_profile.out`](diffhunk://#diff-ee440c0dc7797173bf524be705b22d7ab4044808e19d81825e8ab79b080ad9fcR1-R51): Added new test cases for array inverted index profiles. * [`regression-test/suites/variant_p0/with_index/test_array_contains_with_inverted_index.groovy`](diffhunk://#diff-4fcc39383edd29ad28d9830c401188206c36b23949f831c6cbceb53309ca8b21R1-R112): Introduced a new test suite to validate the `array_contains` function with inverted indexes. * [`regression-test/suites/variant_p0/with_index/test_array_index_write.groovy`](diffhunk://#diff-1658d944fdb787eadefa6eaff841326f6f68c0fccf58899e9a6a0c631017ab63R1-R93): Added tests to verify writing to tables with array inverted indexes and handling various data types. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org
