Github user toyboxman commented on a diff in the pull request:

    https://github.com/apache/griffin/pull/468#discussion_r242442619
  
    --- Diff: 
service/src/main/java/org/apache/griffin/core/job/JobServiceImpl.java ---
    @@ -83,14 +83,7 @@ Licensed to the Apache Software Foundation (ASF) under 
one
     import static 
org.apache.griffin.core.exception.GriffinExceptionMessage.MEASURE_TYPE_DOES_NOT_SUPPORT;
     import static 
org.apache.griffin.core.exception.GriffinExceptionMessage.NO_SUCH_JOB_ACTION;
     import static 
org.apache.griffin.core.exception.GriffinExceptionMessage.QUARTZ_JOB_ALREADY_EXIST;
    -import static 
org.apache.griffin.core.job.entity.LivySessionStates.State.BUSY;
    -import static 
org.apache.griffin.core.job.entity.LivySessionStates.State.DEAD;
    -import static 
org.apache.griffin.core.job.entity.LivySessionStates.State.IDLE;
    -import static 
org.apache.griffin.core.job.entity.LivySessionStates.State.NOT_STARTED;
    -import static 
org.apache.griffin.core.job.entity.LivySessionStates.State.RECOVERING;
    -import static 
org.apache.griffin.core.job.entity.LivySessionStates.State.RUNNING;
    -import static 
org.apache.griffin.core.job.entity.LivySessionStates.State.STARTING;
    -import static 
org.apache.griffin.core.job.entity.LivySessionStates.State.UNKNOWN;
    +import static org.apache.griffin.core.job.entity.LivySessionStates.State.*;
    --- End diff --
    
    please follow 
https://github.com/apache/griffin/blob/master/griffin-doc/dev/code-style.md
    
    remove * from import statement


---

Reply via email to