Gabriel39 opened a new pull request, #65544:
URL: https://github.com/apache/doris/pull/65544

   ### What problem does this PR solve?
   
   Fix a BE UT compile failure introduced by the combination of #65500 and 
#65503.
   `SplitReadOptions` now has `conjuncts` and `all_runtime_filters_applied`, 
while
   the new `jni_table_reader_test.cpp` designated initializer did not initialize
   them. With `-Werror,-Wmissing-designated-field-initializers`, this fails
   compilation.
   
   This PR explicitly initializes the two fields in the test and adds the 
missing
   `<optional>` include for `std::nullopt`.
   
   ### Release note
   
   None
   
   ### Check List (For Author)
   
   - Test: Unit Test
       - `./run-be-ut.sh --run --filter=JniTableReaderTest.*` on 
`[email protected]`
         is running in 
`/mnt/disk3/gabriel/tmp/doris-fix-jni-split-read-options`;
         result will be added after completion.
   - Behavior changed: No
   - Does this need documentation: No
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to