[GitHub] carbondata pull request #2406: [CARBONDATA-2642] Added configurable Lock pat...

[gvramana]

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

    https://github.com/apache/carbondata/pull/2406#discussion_r200246477
  
    --- Diff: 
processing/src/test/java/org/apache/carbondata/lcm/locks/LocalFileLockTest.java 
---
    @@ -68,4 +80,18 @@
         Assert.assertTrue(localLock2.unlock());
       }
     
    +  @Test public void testConfigurablePathForLock() throws Exception {
    +    Field f = secretClass.getDeclaredField("lockPath");
    +    f.setAccessible(true);
    +    f.set(secretClass, rootPath + "/target/");
    +    AbsoluteTableIdentifier absoluteTableIdentifier = 
AbsoluteTableIdentifier
    +        
.from(CarbonProperties.getInstance().getProperty("carbon.storelocation"), 
"databaseName",
    +            "tableName", "1");
    --- End diff --
    
    reset the lockpath in testcase finally clause


---