Skip to content

Make all BatchUtils variables to be final #7215

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
+8 −8
Closed

Make all BatchUtils variables to be final #7215

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 8 additions & 8 deletions kyuubi-rest-client/src/main/java/org/apache/kyuubi/client/util/BatchUtils.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,26 +24,26 @@

public final class BatchUtils {
/** The batch has not been submitted to resource manager yet. */
public static String PENDING_STATE = "PENDING";
public static final String PENDING_STATE = "PENDING";

/** The batch has been submitted to resource manager and is running. */
public static String RUNNING_STATE = "RUNNING";
public static final String RUNNING_STATE = "RUNNING";

/** The batch has been finished successfully. */
public static String FINISHED_STATE = "FINISHED";
public static final String FINISHED_STATE = "FINISHED";

/** The batch met some issue and failed. */
public static String ERROR_STATE = "ERROR";
public static final String ERROR_STATE = "ERROR";

/** The batch was closed by `DELETE /batches/${batchId}` api. */
public static String CANCELED_STATE = "CANCELED";
public static final String CANCELED_STATE = "CANCELED";

public static List<String> terminalBatchStates =
public static final List<String> terminalBatchStates =
Arrays.asList(FINISHED_STATE, ERROR_STATE, CANCELED_STATE);

public static String KYUUBI_BATCH_ID_KEY = "kyuubi.batch.id";
public static final String KYUUBI_BATCH_ID_KEY = "kyuubi.batch.id";

public static String KYUUBI_BATCH_DUPLICATED_KEY = "kyuubi.batch.duplicated";
public static final String KYUUBI_BATCH_DUPLICATED_KEY = "kyuubi.batch.duplicated";

public static boolean isPendingState(String state) {
return PENDING_STATE.equalsIgnoreCase(state);
Expand Down
Loading