Uploaded image for project: 'Hadoop HDFS'
  1. Hadoop HDFS
  2. HDFS-5256

Use guava LoadingCache to implement DFSClientCache

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.2.0
    • Component/s: nfs
    • Labels:
      None

      Description

      Google Guava provides an implementation of LoadingCache. Use the LoadingCache to implement DFSClientCache in NFS.

      1. HDFS-5256.000.patch
        9 kB
        Haohui Mai
      2. HDFS-5256.001.patch
        9 kB
        Haohui Mai
      3. HDFS-5256.002.patch
        10 kB
        Haohui Mai
      4. HDFS-5256.003.patch
        10 kB
        Haohui Mai
      5. HDFS-5256.004.patch
        10 kB
        Haohui Mai

        Activity

        Hide
        hadoopqa Hadoop QA added a comment -

        +1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12604926/HDFS-5256.000.patch
        against trunk revision .

        +1 @author. The patch does not contain any @author tags.

        +1 tests included. The patch appears to include 1 new or modified test files.

        +1 javac. The applied patch does not increase the total number of javac compiler warnings.

        +1 javadoc. The javadoc tool did not generate any warning messages.

        +1 eclipse:eclipse. The patch built with eclipse:eclipse.

        +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) warnings.

        +1 release audit. The applied patch does not increase the total number of release audit warnings.

        +1 core tests. The patch passed unit tests in hadoop-hdfs-project/hadoop-hdfs-nfs.

        +1 contrib tests. The patch passed contrib unit tests.

        Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/5031//testReport/
        Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/5031//console

        This message is automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - +1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12604926/HDFS-5256.000.patch against trunk revision . +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 1 new or modified test files. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . The javadoc tool did not generate any warning messages. +1 eclipse:eclipse . The patch built with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 1.3.9) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. +1 core tests . The patch passed unit tests in hadoop-hdfs-project/hadoop-hdfs-nfs. +1 contrib tests . The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/5031//testReport/ Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/5031//console This message is automatically generated.
        Hide
        wheat9 Haohui Mai added a comment -

        Address Jing's comment + plus a little bit of formatting.

        Show
        wheat9 Haohui Mai added a comment - Address Jing's comment + plus a little bit of formatting.
        Hide
        wheat9 Haohui Mai added a comment -

        Address Brandon's comments. More comprehensive unit tests.

        Show
        wheat9 Haohui Mai added a comment - Address Brandon's comments. More comprehensive unit tests.
        Hide
        hadoopqa Hadoop QA added a comment -

        +1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12605309/HDFS-5256.001.patch
        against trunk revision .

        +1 @author. The patch does not contain any @author tags.

        +1 tests included. The patch appears to include 1 new or modified test files.

        +1 javac. The applied patch does not increase the total number of javac compiler warnings.

        +1 javadoc. The javadoc tool did not generate any warning messages.

        +1 eclipse:eclipse. The patch built with eclipse:eclipse.

        +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) warnings.

        +1 release audit. The applied patch does not increase the total number of release audit warnings.

        +1 core tests. The patch passed unit tests in hadoop-hdfs-project/hadoop-hdfs-nfs.

        +1 contrib tests. The patch passed contrib unit tests.

        Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/5047//testReport/
        Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/5047//console

        This message is automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - +1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12605309/HDFS-5256.001.patch against trunk revision . +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 1 new or modified test files. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . The javadoc tool did not generate any warning messages. +1 eclipse:eclipse . The patch built with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 1.3.9) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. +1 core tests . The patch passed unit tests in hadoop-hdfs-project/hadoop-hdfs-nfs. +1 contrib tests . The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/5047//testReport/ Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/5047//console This message is automatically generated.
        Hide
        brandonli Brandon Li added a comment -

        Thanks Haohui for adding unit test. More ideas for better unit tests of the new cache:
        1. add n clients to the cache, check the cache size is n, remove some client, and make sure the cache size changes correspondingly
        2. do cache.get() for the same user, make sure the cache doesn't add new entry for the same user
        3. make sure the evicted client is closed (which you are already trying to test with current patch)
        4. give the cache a short expiration time to verify the client can be evicted and closed after expiration time

        Show
        brandonli Brandon Li added a comment - Thanks Haohui for adding unit test. More ideas for better unit tests of the new cache: 1. add n clients to the cache, check the cache size is n, remove some client, and make sure the cache size changes correspondingly 2. do cache.get() for the same user, make sure the cache doesn't add new entry for the same user 3. make sure the evicted client is closed (which you are already trying to test with current patch) 4. give the cache a short expiration time to verify the client can be evicted and closed after expiration time
        Hide
        wheat9 Haohui Mai added a comment -

        Addressed Brandon's comments.

        Show
        wheat9 Haohui Mai added a comment - Addressed Brandon's comments.
        Hide
        hadoopqa Hadoop QA added a comment -

        +1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12605321/HDFS-5256.002.patch
        against trunk revision .

        +1 @author. The patch does not contain any @author tags.

        +1 tests included. The patch appears to include 1 new or modified test files.

        +1 javac. The applied patch does not increase the total number of javac compiler warnings.

        +1 javadoc. The javadoc tool did not generate any warning messages.

        +1 eclipse:eclipse. The patch built with eclipse:eclipse.

        +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) warnings.

        +1 release audit. The applied patch does not increase the total number of release audit warnings.

        +1 core tests. The patch passed unit tests in hadoop-hdfs-project/hadoop-hdfs-nfs.

        +1 contrib tests. The patch passed contrib unit tests.

        Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/5048//testReport/
        Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/5048//console

        This message is automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - +1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12605321/HDFS-5256.002.patch against trunk revision . +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 1 new or modified test files. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . The javadoc tool did not generate any warning messages. +1 eclipse:eclipse . The patch built with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 1.3.9) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. +1 core tests . The patch passed unit tests in hadoop-hdfs-project/hadoop-hdfs-nfs. +1 contrib tests . The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/5048//testReport/ Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/5048//console This message is automatically generated.
        Hide
        hadoopqa Hadoop QA added a comment -

        +1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12605324/HDFS-5256.003.patch
        against trunk revision .

        +1 @author. The patch does not contain any @author tags.

        +1 tests included. The patch appears to include 1 new or modified test files.

        +1 javac. The applied patch does not increase the total number of javac compiler warnings.

        +1 javadoc. The javadoc tool did not generate any warning messages.

        +1 eclipse:eclipse. The patch built with eclipse:eclipse.

        +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) warnings.

        +1 release audit. The applied patch does not increase the total number of release audit warnings.

        +1 core tests. The patch passed unit tests in hadoop-hdfs-project/hadoop-hdfs-nfs.

        +1 contrib tests. The patch passed contrib unit tests.

        Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/5050//testReport/
        Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/5050//console

        This message is automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - +1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12605324/HDFS-5256.003.patch against trunk revision . +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 1 new or modified test files. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . The javadoc tool did not generate any warning messages. +1 eclipse:eclipse . The patch built with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 1.3.9) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. +1 core tests . The patch passed unit tests in hadoop-hdfs-project/hadoop-hdfs-nfs. +1 contrib tests . The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/5050//testReport/ Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/5050//console This message is automatically generated.
        Hide
        brandonli Brandon Li added a comment -

        Haohui, let's remove the TTL since the cache entry usage inside NFS servers may not actually visible here. Also please make this warning message more verbose in DFSClientCache.java, such as:
        LOG.warn("DFSClientCache got IOException when closing the DFSClient("
        + notification.getValue() + "): " + e);

        Show
        brandonli Brandon Li added a comment - Haohui, let's remove the TTL since the cache entry usage inside NFS servers may not actually visible here. Also please make this warning message more verbose in DFSClientCache.java, such as: LOG.warn("DFSClientCache got IOException when closing the DFSClient(" + notification.getValue() + "): " + e);
        Hide
        wheat9 Haohui Mai added a comment -

        Thanks Brandon Li for the comments. Address the comments.

        Show
        wheat9 Haohui Mai added a comment - Thanks Brandon Li for the comments. Address the comments.
        Hide
        hadoopqa Hadoop QA added a comment -

        -1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12605382/HDFS-5256.004.patch
        against trunk revision .

        +1 @author. The patch does not contain any @author tags.

        +1 tests included. The patch appears to include 1 new or modified test files.

        +1 javac. The applied patch does not increase the total number of javac compiler warnings.

        +1 javadoc. The javadoc tool did not generate any warning messages.

        -1 eclipse:eclipse. The patch failed to build with eclipse:eclipse.

        +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) warnings.

        +1 release audit. The applied patch does not increase the total number of release audit warnings.

        +1 core tests. The patch passed unit tests in hadoop-hdfs-project/hadoop-hdfs-nfs.

        +1 contrib tests. The patch passed contrib unit tests.

        Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/5051//testReport/
        Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/5051//console

        This message is automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12605382/HDFS-5256.004.patch against trunk revision . +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 1 new or modified test files. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . The javadoc tool did not generate any warning messages. -1 eclipse:eclipse . The patch failed to build with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 1.3.9) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. +1 core tests . The patch passed unit tests in hadoop-hdfs-project/hadoop-hdfs-nfs. +1 contrib tests . The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/5051//testReport/ Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/5051//console This message is automatically generated.
        Hide
        brandonli Brandon Li added a comment -

        +1. Thank you, Haohui, for refining the patch's details!

        Show
        brandonli Brandon Li added a comment - +1. Thank you, Haohui, for refining the patch's details!
        Hide
        brandonli Brandon Li added a comment -

        I've committed the patch. Thank you, Haohui, for the contribution!

        Show
        brandonli Brandon Li added a comment - I've committed the patch. Thank you, Haohui, for the contribution!
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Hadoop-trunk-Commit #4496 (See https://builds.apache.org/job/Hadoop-trunk-Commit/4496/)
        HDFS-5256. Use guava LoadingCache to implement DFSClientCache. Contributed by Haohui Mai (brandonli: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1527452)

        • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-nfs/src/main/java/org/apache/hadoop/hdfs/nfs/nfs3/DFSClientCache.java
        • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-nfs/src/main/java/org/apache/hadoop/hdfs/nfs/nfs3/LruCache.java
        • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-nfs/src/test/java/org/apache/hadoop/hdfs/nfs/nfs3/TestDFSClientCache.java
        • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Hadoop-trunk-Commit #4496 (See https://builds.apache.org/job/Hadoop-trunk-Commit/4496/ ) HDFS-5256 . Use guava LoadingCache to implement DFSClientCache. Contributed by Haohui Mai (brandonli: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1527452 ) /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-nfs/src/main/java/org/apache/hadoop/hdfs/nfs/nfs3/DFSClientCache.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-nfs/src/main/java/org/apache/hadoop/hdfs/nfs/nfs3/LruCache.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-nfs/src/test/java/org/apache/hadoop/hdfs/nfs/nfs3/TestDFSClientCache.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Hadoop-Yarn-trunk #348 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/348/)
        HDFS-5256. Use guava LoadingCache to implement DFSClientCache. Contributed by Haohui Mai (brandonli: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1527452)

        • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-nfs/src/main/java/org/apache/hadoop/hdfs/nfs/nfs3/DFSClientCache.java
        • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-nfs/src/main/java/org/apache/hadoop/hdfs/nfs/nfs3/LruCache.java
        • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-nfs/src/test/java/org/apache/hadoop/hdfs/nfs/nfs3/TestDFSClientCache.java
        • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Yarn-trunk #348 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/348/ ) HDFS-5256 . Use guava LoadingCache to implement DFSClientCache. Contributed by Haohui Mai (brandonli: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1527452 ) /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-nfs/src/main/java/org/apache/hadoop/hdfs/nfs/nfs3/DFSClientCache.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-nfs/src/main/java/org/apache/hadoop/hdfs/nfs/nfs3/LruCache.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-nfs/src/test/java/org/apache/hadoop/hdfs/nfs/nfs3/TestDFSClientCache.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Hadoop-Hdfs-trunk #1538 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/1538/)
        HDFS-5256. Use guava LoadingCache to implement DFSClientCache. Contributed by Haohui Mai (brandonli: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1527452)

        • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-nfs/src/main/java/org/apache/hadoop/hdfs/nfs/nfs3/DFSClientCache.java
        • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-nfs/src/main/java/org/apache/hadoop/hdfs/nfs/nfs3/LruCache.java
        • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-nfs/src/test/java/org/apache/hadoop/hdfs/nfs/nfs3/TestDFSClientCache.java
        • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Hadoop-Hdfs-trunk #1538 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/1538/ ) HDFS-5256 . Use guava LoadingCache to implement DFSClientCache. Contributed by Haohui Mai (brandonli: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1527452 ) /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-nfs/src/main/java/org/apache/hadoop/hdfs/nfs/nfs3/DFSClientCache.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-nfs/src/main/java/org/apache/hadoop/hdfs/nfs/nfs3/LruCache.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-nfs/src/test/java/org/apache/hadoop/hdfs/nfs/nfs3/TestDFSClientCache.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Hadoop-Mapreduce-trunk #1564 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1564/)
        HDFS-5256. Use guava LoadingCache to implement DFSClientCache. Contributed by Haohui Mai (brandonli: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1527452)

        • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-nfs/src/main/java/org/apache/hadoop/hdfs/nfs/nfs3/DFSClientCache.java
        • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-nfs/src/main/java/org/apache/hadoop/hdfs/nfs/nfs3/LruCache.java
        • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-nfs/src/test/java/org/apache/hadoop/hdfs/nfs/nfs3/TestDFSClientCache.java
        • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Mapreduce-trunk #1564 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1564/ ) HDFS-5256 . Use guava LoadingCache to implement DFSClientCache. Contributed by Haohui Mai (brandonli: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1527452 ) /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-nfs/src/main/java/org/apache/hadoop/hdfs/nfs/nfs3/DFSClientCache.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-nfs/src/main/java/org/apache/hadoop/hdfs/nfs/nfs3/LruCache.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-nfs/src/test/java/org/apache/hadoop/hdfs/nfs/nfs3/TestDFSClientCache.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
        Hide
        eli Eli Collins added a comment -
        • How does testEviction test eviction? It sets a max cache size of 2 and only ever caches one entry?
        • Nit: clientCache is overloaded as both an integer and a LoadingCache object in the constructor
        • Nit: Spelling mistake in "clientRemovealListener"
        Show
        eli Eli Collins added a comment - How does testEviction test eviction? It sets a max cache size of 2 and only ever caches one entry? Nit: clientCache is overloaded as both an integer and a LoadingCache object in the constructor Nit: Spelling mistake in "clientRemovealListener"
        Hide
        wheat9 Haohui Mai added a comment -

        How does testEviction test eviction? It sets a max cache size of 2 and only ever caches one entry?

        The guava cache only holds MAX_CACHE_SIZE - 1 entries, therefore the unit test is a basic test whether the eviction works.

        The unit test is basically a rewrite of the legacy unit test. We used to implement the cache in house, but
        given the fact that DFSClientCache is a thin wrap of guava cache, the test looks more like a unit test of the guava cache rather than a test of DFSClientCache.

        Nit: clientCache is overloaded as both an integer and a LoadingCache object in the constructor

        Testing is the only reason of implementing two constructors of DFSClientCache(). It should be cleaned up once we reach a consensus on the unit test.

        Nit: Spelling mistake in "clientRemovealListener"

        I'll fix it in a separate jira.

        Show
        wheat9 Haohui Mai added a comment - How does testEviction test eviction? It sets a max cache size of 2 and only ever caches one entry? The guava cache only holds MAX_CACHE_SIZE - 1 entries, therefore the unit test is a basic test whether the eviction works. The unit test is basically a rewrite of the legacy unit test. We used to implement the cache in house, but given the fact that DFSClientCache is a thin wrap of guava cache, the test looks more like a unit test of the guava cache rather than a test of DFSClientCache. Nit: clientCache is overloaded as both an integer and a LoadingCache object in the constructor Testing is the only reason of implementing two constructors of DFSClientCache(). It should be cleaned up once we reach a consensus on the unit test. Nit: Spelling mistake in "clientRemovealListener" I'll fix it in a separate jira.
        Hide
        vinodkv Vinod Kumar Vavilapalli added a comment -

        Closing tickets that are already part of a release.

        Show
        vinodkv Vinod Kumar Vavilapalli added a comment - Closing tickets that are already part of a release.

          People

          • Assignee:
            wheat9 Haohui Mai
            Reporter:
            wheat9 Haohui Mai
          • Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development