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

    https://github.com/apache/carbondata/pull/1702#discussion_r158480429
  
    --- Diff: 
integration/spark2/src/main/scala/org/apache/carbondata/spark/rdd/CarbonDataRDDFactory.scala
 ---
    @@ -418,8 +429,11 @@ object CarbonDataRDDFactory {
                   errorMessage = errorMessage + ": " + executorMessage
                 }
             }
    +        segmentLock.unlock()
    --- End diff --
    
    Please remove this. unlock in finally is enough


---

Reply via email to