nsivabalan commented on code in PR #9611:
URL: https://github.com/apache/hudi/pull/9611#discussion_r1320485055
##########
hudi-client/hudi-flink-client/src/main/java/org/apache/hudi/client/FlinkTaskContextSupplier.java:
##########
@@ -62,4 +62,9 @@ public Option<String> getProperty(EngineProperty prop) {
return Option.empty();
}
+ @Override
+ public Supplier<Integer> getAttemptNumberSupplier() {
+ return () -> -1;
Review Comment:
yes. we have disabled it for flink as of now. In java, anyways, MOR is not
fully functional from what I know. but I can open to disabling it for java as
well. mainly its an issue for ExpressionPayload and any other custom payloads.
most of the other payloads are idempotent even if there are duplicate log
blocks.
--
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]
