This is an automated email from the ASF dual-hosted git repository.
hello-stephen pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new a37667de5ae [fix](be) Initialize JNI split read options in test
(#65544)
a37667de5ae is described below
commit a37667de5ae2cddb2be6aace2a80289ce5ec34fc
Author: Gabriel <[email protected]>
AuthorDate: Mon Jul 13 19:32:58 2026 +0800
[fix](be) Initialize JNI split read options in test (#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`.
---
be/test/format_v2/jni/jni_table_reader_test.cpp | 3 +++
1 file changed, 3 insertions(+)
diff --git a/be/test/format_v2/jni/jni_table_reader_test.cpp
b/be/test/format_v2/jni/jni_table_reader_test.cpp
index 1c5788b1df7..db3ec3c7e02 100644
--- a/be/test/format_v2/jni/jni_table_reader_test.cpp
+++ b/be/test/format_v2/jni/jni_table_reader_test.cpp
@@ -21,6 +21,7 @@
#include <map>
#include <memory>
+#include <optional>
#include <string>
#include <vector>
@@ -175,7 +176,9 @@ TEST(JniTableReaderTest,
AdaptiveProbeSetBeforePrepareControlsFirstJniOpen) {
reader.set_batch_size(32);
ASSERT_TRUE(reader.prepare_split({
.partition_values = {},
+ .conjuncts = std::nullopt,
.partition_prune_conjuncts = {},
+ .all_runtime_filters_applied =
true,
.cache = nullptr,
.current_range = {},
.current_split_format =
FileFormat::JNI,
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
