JingsongLi commented on code in PR #898:
URL: https://github.com/apache/incubator-paimon/pull/898#discussion_r1165369699


##########
paimon-core/src/main/java/org/apache/paimon/operation/FileStoreCommit.java:
##########
@@ -72,4 +73,7 @@ void overwrite(
      * @param partitions A list of partition {@link Map}s. NOTE: cannot be 
empty!
      */
     void dropPartitions(List<Map<String, String>> partitions, long 
commitIdentifier);
+
+    /** Abort an unsuccessful commit. The data files will be deleted. */
+    void abort(List<CommitMessage> commitMessages);

Review Comment:
   `ManifestCommittable` need an identifier. But here we don't need that.



-- 
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]

Reply via email to