924060929 commented on code in PR #67687:
URL: https://github.com/apache/doris/pull/67687#discussion_r4001756920
##########
fe/pom.xml:
##########
@@ -360,7 +360,7 @@ under the License.
<!-- ATTN: avro version must be consistent with Iceberg version -->
<!-- Please modify iceberg.version and avro.version together,
you can find avro version info in iceberg mvn repository -->
- <iceberg.version>1.10.1</iceberg.version>
+ <iceberg.version>1.11.0</iceberg.version>
Review Comment:
After tracing the failure through the JNI boundary, I agree this
compatibility gap should not block the PR. For the concrete BE-first mismatch
discussed here, deserializing the Iceberg 1.10.1 `$all_manifests` task on the
1.11.0 BE throws during scanner construction; JNI clears the Java exception and
converts it to a non-OK `JniError`, and `FileScanner` fails the affected query.
It does not crash the BE, return incorrect rows, corrupt data, or affect
ordinary Iceberg scans. The mismatch also disappears after the FE is upgraded.
Given the narrow and temporary impact, I consider this a non-blocking
rolling-upgrade limitation rather than a P1 issue.
--
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]
