[GitHub] metron pull request #1157: METRON-1732: Fix job status liveness bug and para...

[nickwallen]

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

    https://github.com/apache/metron/pull/1157#discussion_r209665613
  
    --- Diff: 
metron-platform/metron-pcap/src/main/java/org/apache/metron/pcap/mr/PcapJob.java
 ---
    @@ -307,8 +307,11 @@ public void setCompleteCheckInterval(long interval) {
           }
           return this;
         }
    -    mrJob.submit();
    -    jobStatus.withState(State.SUBMITTED).withDescription("Job 
submitted").withJobId(mrJob.getJobID().toString());
    +    synchronized (this) {
    --- End diff --
    
    Can we add a comment about why we need the lock here?


---