package org.apache.solr.store.hdfs;

/*
 * Licensed to the Apache Software Foundation (ASF) under one or more
 * contributor license agreements.  See the NOTICE file distributed with
 * this work for additional information regarding copyright ownership.
 * The ASF licenses this file to You under the Apache License, Version 2.0
 * (the "License"); you may not use this file except in compliance with
 * the License.  You may obtain a copy of the License at
 *
 *     http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */

import java.io.IOException;
import java.net.URI;

import org.apache.hadoop.conf.Configuration;
import org.apache.hadoop.fs.Path;
import org.apache.hadoop.hdfs.MiniDFSCluster;
import org.apache.lucene.store.Lock;
import org.apache.lucene.store.LockObtainFailedException;
import org.apache.solr.SolrTestCaseJ4;
import org.apache.solr.cloud.hdfs.HdfsTestUtil;
import org.apache.solr.util.BadHdfsThreadsFilter;
import org.junit.After;
import org.junit.AfterClass;
import org.junit.Before;
import org.junit.BeforeClass;
import org.junit.Test;
import com.carrotsearch.randomizedtesting.annotations.ThreadLeakFilters;

@ThreadLeakFilters(defaultFilters = true, filters = {
    BadHdfsThreadsFilter.class // hdfs currently leaks thread(s)
})
public class HdfsLockFactoryTest extends SolrTestCaseJ4 {
  
  private static MiniDFSCluster dfsCluster;

  @BeforeClass
  public static void beforeClass() throws Exception {
    dfsCluster = HdfsTestUtil.setupClass(createTempDir().toFile().getAbsolutePath());
  }

  @AfterClass
  public static void afterClass() throws Exception {
    HdfsTestUtil.teardownClass(dfsCluster);
    dfsCluster = null;
  }
  
  @Before
  public void setUp() throws Exception {
    super.setUp();
  }
  
  @After
  public void tearDown() throws Exception {
    super.tearDown();
  }
  
  @Test
  public void testBasic() throws IOException {
    String uri = HdfsTestUtil.getURI(dfsCluster);
    Path lockPath = new Path(uri, "/basedir/lock");
    Configuration conf = HdfsTestUtil.getClientConfiguration(dfsCluster);
    HdfsDirectory dir = new HdfsDirectory(lockPath, conf);
    Lock lock = dir.makeLock("testlock");
    boolean success = lock.obtain();
    assertTrue("We could not get the lock when it should be available", success);
    success = lock.obtain();
    assertFalse("We got the lock but it should be unavailble", success);
    lock.close();
    success = lock.obtain();
    assertTrue("We could not get the lock when it should be available", success);
    success = lock.obtain();
    assertFalse("We got the lock but it should be unavailble", success);
    dir.close();
  }
  
  public void testDoubleObtain() throws Exception {
    String uri = HdfsTestUtil.getURI(dfsCluster);
    Path lockPath = new Path(uri, "/basedir/lock");
    Configuration conf = HdfsTestUtil.getClientConfiguration(dfsCluster);
    HdfsDirectory dir = new HdfsDirectory(lockPath, conf);
    Lock lock = dir.makeLock("foo");
    assertTrue(lock.obtain());
    try {
      lock.obtain();
      fail("did not hit double-obtain failure");
    } catch (LockObtainFailedException lofe) {
      // expected
    }
    lock.close();
    
    lock = dir.makeLock("foo");
    assertTrue(lock.obtain());
    lock.close();
    dir.close();
  }
}

@@ -1099,24 +1099,63 @@
 dException;%0A
+import org.apache.lucene.util.IOUtils;%0A
 import org.a
@@ -1410,16 +1410,17 @@
 t.Test;%0A
+%0A
 import c
@@ -2616,32 +2616,75 @@
 ble%22, success);%0A
+    Lock lock2 = dir.makeLock(%22testlock%22);%0A
     success = lo
@@ -2677,32 +2677,33 @@
   success = lock
+2
 .obtain();%0A    a
@@ -2770,35 +2770,118 @@
 ccess);%0A    
-lock.close(
+IOUtils.close(lock, lock2);%0A    // now repeat after close()%0A    lock = dir.makeLock(%22testlock%22
 );%0A    succe
@@ -2978,24 +2978,62 @@
 , success);%0A
+    lock2 = dir.makeLock(%22testlock%22);%0A
     success 
@@ -3030,32 +3030,33 @@
   success = lock
+2
 .obtain();%0A    a
@@ -3119,24 +3119,56 @@
 , success);%0A
+    IOUtils.close(lock, lock2);%0A
     dir.clos

fix test bug to not double-obtain. testDoubleObtain already tests that