dongtingting commented on issue #11516: URL: https://github.com/apache/hudi/issues/11516#issuecomment-2242990082
> > TransactionUtils.resolveWriteConflictIfAny getCandidateInstants function want to get pending clustering. Now get pending clustering instants by REPLACE_COMMIT_ACTION, actually here get both clustering and insertoverwrite instants. > > So, I think we can continue filter by get clusteringplan is present ， it will get only clustering instant。 > > Yeah, for your use case there is no need to resolve conflicts for `insert_overwrite` because it occurs in different partitions, but from Hudi side, we still need to take it into consideration because there is no prompt to tell Hudi that the ingestion and `insert_overwrite` happens in different partitions(only after a resolution of conflict we can find it out). > > It looks like you have fond the culprit of the `StringIndexOutOfBoundsException`, would you mind to fire a fix for it? Yes, I am very glad to fix it. Allow me to further clarify （forgive my repetition ）, i also think it is need to resolve conflicts for `insert_overwrite`. when `insert_overwrite` instant is complete, it will be checked. Can I fix it like following picture？do you agree？ <img width="2033" alt="image" src="https://github.com/user-attachments/assets/a12514e6-6a93-4e5d-83ea-bd3cac000d9b";> -- 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: commits-unsubscr...@hudi.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
