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:
For this rolling-upgrade policy, a short period of query failures is
acceptable as long as the incompatibility cannot crash the BE or return
incorrect results. The concrete 1.10.1-to-1.11.0 `$all_manifests` mismatch
meets that bar: JNI clears the Java deserialization exception and converts it
to a non-OK `JniError`, so only the affected query fails during the
BE-first/FE-second window. It does not cause a BE core dump, return wrong rows,
or corrupt data, and the query recovers after the FE is upgraded. Therefore
this compatibility gap is non-blocking.
--
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]
