morningman commented on a change in pull request #2241: Fix some bugs about
load label
URL: https://github.com/apache/incubator-doris/pull/2241#discussion_r349391023
##########
File path: fe/src/main/java/org/apache/doris/load/loadv2/JobState.java
##########
@@ -18,8 +18,9 @@
package org.apache.doris.load.loadv2;
public enum JobState {
- PENDING,
- LOADING,
- FINISHED,
- CANCELLED
+ PENDING, // init state
+ LOADING, // job is running
+ COMMITTED, // transaction is committed but not visible
Review comment:
we persist state by name, not by order, so it is OK.
And most of enum are persist by name, which is a common usage.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
