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

    https://github.com/apache/carbondata/pull/2796#discussion_r222637804
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/util/ThreadLocalTaskInfo.java ---
    @@ -30,7 +32,7 @@ public static void setCarbonTaskInfo(CarbonTaskInfo 
carbonTaskInfo) {
       public static CarbonTaskInfo getCarbonTaskInfo() {
         if (null == threadLocal.get()) {
           CarbonTaskInfo carbonTaskInfo = new CarbonTaskInfo();
    -      carbonTaskInfo.setTaskId(System.nanoTime());
    +      carbonTaskInfo.setTaskId(UUID.randomUUID().toString());
    --- End diff --
    
    Please move the generation of UUID to some UUID to some common utility so 
all methods can use same. In future if we need to change we can change at one 
place


---

Reply via email to