Hadoop HDFS
  1. Hadoop HDFS
  2. HDFS-6564

Use slf4j instead of common-logging in hdfs-client

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.8.0
    • Component/s: None
    • Labels:
      None
    • Hadoop Flags:
      Incompatible change, Reviewed
    • Release Note:
      Hide
      Users may need special attention for this change while upgrading to this version. Previously hdfs client was using commons-logging as the logging framework. With this change it will use slf4j framework. For more details about slf4j, please see: http://www.slf4j.org/manual.html. Also, org.apache.hadoop.hdfs.protocol.CachePoolInfo#LOG public static member variable has been removed as it is not used anywhere. Users need to correct their code if any one has a reference to this variable. One can retrieve the named logger via the logging framework of their choice directly like, org.slf4j.Logger LOG = org.slf4j.LoggerFactory.getLogger(org.apache.hadoop.hdfs.protocol.CachePoolInfo.class);
      Show
      Users may need special attention for this change while upgrading to this version. Previously hdfs client was using commons-logging as the logging framework. With this change it will use slf4j framework. For more details about slf4j, please see: http://www.slf4j.org/manual.html . Also, org.apache.hadoop.hdfs.protocol.CachePoolInfo#LOG public static member variable has been removed as it is not used anywhere. Users need to correct their code if any one has a reference to this variable. One can retrieve the named logger via the logging framework of their choice directly like, org.slf4j.Logger LOG = org.slf4j.LoggerFactory.getLogger(org.apache.hadoop.hdfs.protocol.CachePoolInfo.class);

      Description

      hdfs-client should depends on slf4j instead of common-logging.

      1. HDFS-6564-01.patch
        15 kB
        Rakesh R
      2. HDFS-6564-02.patch
        14 kB
        Rakesh R
      3. HDFS-6564-03.patch
        14 kB
        Rakesh R

        Issue Links

          Activity

          Haohui Mai created issue -
          Haohui Mai made changes -
          Field Original Value New Value
          Link This issue depends upon HDFS-6200 [ HDFS-6200 ]
          Rakesh R made changes -
          Attachment HDFS-6564-01.patch [ 12740092 ]
          Hide
          Rakesh R added a comment -

          Hi Haohui Mai, its interesting work. I have attached a patch to address the case. Please take a look when you get a chance. Thanks!

          Show
          Rakesh R added a comment - Hi Haohui Mai , its interesting work. I have attached a patch to address the case. Please take a look when you get a chance. Thanks!
          Rakesh R made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Rakesh R made changes -
          Assignee Rakesh R [ rakeshr ]
          Hide
          Sean Busbey added a comment -
          diff --git hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/CachePoolInfo.java hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/CachePoolInfo.java
          index 61bbe38..31850dc 100644
          --- hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/CachePoolInfo.java
          +++ hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/CachePoolInfo.java
          @@ -24,13 +24,13 @@
           
           import org.apache.commons.lang.builder.EqualsBuilder;
           import org.apache.commons.lang.builder.HashCodeBuilder;
          -import org.apache.commons.logging.Log;
          -import org.apache.commons.logging.LogFactory;
           import org.apache.hadoop.classification.InterfaceAudience;
           import org.apache.hadoop.classification.InterfaceStability;
           import org.apache.hadoop.fs.InvalidRequestException;
           import org.apache.hadoop.fs.permission.FsPermission;
           import org.apache.hadoop.hdfs.protocol.CacheDirectiveInfo.Expiration;
          +import org.slf4j.Logger;
          +import org.slf4j.LoggerFactory;
           
           /**
            * CachePoolInfo describes a cache pool.
          @@ -41,7 +41,8 @@
           @InterfaceAudience.Public
           @InterfaceStability.Evolving
           public class CachePoolInfo {
          -  public static final Log LOG = LogFactory.getLog(CachePoolInfo.class);
          +  public static final Logger LOG = LoggerFactory
          +      .getLogger(CachePoolInfo.class);
          

          This is a binary and source incompatible change to public API. Please be sure to flag the jira as such and provide release notes.

          @@ -733,9 +723,7 @@ final T getResponse(HttpURLConnection conn) throws IOException {
                 } catch (Exception e) { // catch json parser errors
                   final IOException ioe =
                       new IOException("Response decoding failure: "+e.toString(), e);
          -        if (LOG.isDebugEnabled()) {
          -          LOG.debug(ioe);
          -        }
          +        LOG.debug("Response decoding failure: {}", e.toString(), e);
                   throw ioe;
          

          nit: it's possible, though unlikely, that this e.toString call is still expensive and worth guarding with isDebugEnabled.

          Show
          Sean Busbey added a comment - diff --git hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/CachePoolInfo.java hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/CachePoolInfo.java index 61bbe38..31850dc 100644 --- hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/CachePoolInfo.java +++ hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/CachePoolInfo.java @@ -24,13 +24,13 @@ import org.apache.commons.lang.builder.EqualsBuilder; import org.apache.commons.lang.builder.HashCodeBuilder; - import org.apache.commons.logging.Log; - import org.apache.commons.logging.LogFactory; import org.apache.hadoop.classification.InterfaceAudience; import org.apache.hadoop.classification.InterfaceStability; import org.apache.hadoop.fs.InvalidRequestException; import org.apache.hadoop.fs.permission.FsPermission; import org.apache.hadoop.hdfs.protocol.CacheDirectiveInfo.Expiration; + import org.slf4j.Logger; + import org.slf4j.LoggerFactory; /** * CachePoolInfo describes a cache pool. @@ -41,7 +41,8 @@ @InterfaceAudience.Public @InterfaceStability.Evolving public class CachePoolInfo { - public static final Log LOG = LogFactory.getLog(CachePoolInfo.class); + public static final Logger LOG = LoggerFactory + .getLogger(CachePoolInfo.class); This is a binary and source incompatible change to public API. Please be sure to flag the jira as such and provide release notes. @@ -733,9 +723,7 @@ final T getResponse(HttpURLConnection conn) throws IOException { } catch (Exception e) { // catch json parser errors final IOException ioe = new IOException( "Response decoding failure: " +e.toString(), e); - if (LOG.isDebugEnabled()) { - LOG.debug(ioe); - } + LOG.debug( "Response decoding failure: {}" , e.toString(), e); throw ioe; nit: it's possible, though unlikely, that this e.toString call is still expensive and worth guarding with isDebugEnabled.
          Hide
          Rakesh R added a comment -

          Thanks Sean Busbey for the reviews.

          nit: it's possible, though unlikely, that this e.toString call is still expensive and worth guarding with isDebugEnabled.

          Yes, you are right. I will take care this.

          Show
          Rakesh R added a comment - Thanks Sean Busbey for the reviews. nit: it's possible, though unlikely, that this e.toString call is still expensive and worth guarding with isDebugEnabled. Yes, you are right. I will take care this.
          Hide
          Hadoop QA added a comment -



          -1 overall



          Vote Subsystem Runtime Comment
          0 pre-patch 15m 21s Pre-patch trunk compilation is healthy.
          +1 @author 0m 0s The patch does not contain any @author tags.
          -1 tests included 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
          +1 javac 7m 32s There were no new javac warning messages.
          +1 javadoc 9m 35s There were no new javadoc warning messages.
          +1 release audit 0m 21s The applied patch does not increase the total number of release audit warnings.
          -1 checkstyle 0m 21s The applied patch generated 1 new checkstyle issues (total was 156, now 155).
          +1 whitespace 0m 0s The patch has no lines that end in whitespace.
          +1 install 1m 34s mvn install still works.
          +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse.
          +1 findbugs 0m 51s The patch does not introduce any new Findbugs (version 3.0.0) warnings.
          +1 native 3m 14s Pre-build of native portion
          +1 hdfs tests 0m 16s Tests passed in hadoop-hdfs-client.
              39m 42s  



          Subsystem Report/Notes
          Patch URL http://issues.apache.org/jira/secure/attachment/12740092/HDFS-6564-01.patch
          Optional Tests javadoc javac unit findbugs checkstyle
          git revision trunk / 6e3fcff
          checkstyle https://builds.apache.org/job/PreCommit-HDFS-Build/11391/artifact/patchprocess/diffcheckstylehadoop-hdfs-client.txt
          hadoop-hdfs-client test log https://builds.apache.org/job/PreCommit-HDFS-Build/11391/artifact/patchprocess/testrun_hadoop-hdfs-client.txt
          Test Results https://builds.apache.org/job/PreCommit-HDFS-Build/11391/testReport/
          Java 1.7.0_55
          uname Linux asf902.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
          Console output https://builds.apache.org/job/PreCommit-HDFS-Build/11391/console

          This message was automatically generated.

          Show
          Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 pre-patch 15m 21s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. -1 tests included 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. +1 javac 7m 32s There were no new javac warning messages. +1 javadoc 9m 35s There were no new javadoc warning messages. +1 release audit 0m 21s The applied patch does not increase the total number of release audit warnings. -1 checkstyle 0m 21s The applied patch generated 1 new checkstyle issues (total was 156, now 155). +1 whitespace 0m 0s The patch has no lines that end in whitespace. +1 install 1m 34s mvn install still works. +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse. +1 findbugs 0m 51s The patch does not introduce any new Findbugs (version 3.0.0) warnings. +1 native 3m 14s Pre-build of native portion +1 hdfs tests 0m 16s Tests passed in hadoop-hdfs-client.     39m 42s   Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12740092/HDFS-6564-01.patch Optional Tests javadoc javac unit findbugs checkstyle git revision trunk / 6e3fcff checkstyle https://builds.apache.org/job/PreCommit-HDFS-Build/11391/artifact/patchprocess/diffcheckstylehadoop-hdfs-client.txt hadoop-hdfs-client test log https://builds.apache.org/job/PreCommit-HDFS-Build/11391/artifact/patchprocess/testrun_hadoop-hdfs-client.txt Test Results https://builds.apache.org/job/PreCommit-HDFS-Build/11391/testReport/ Java 1.7.0_55 uname Linux asf902.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Console output https://builds.apache.org/job/PreCommit-HDFS-Build/11391/console This message was automatically generated.
          Rakesh R made changes -
          Attachment HDFS-6564-02.patch [ 12740186 ]
          Hide
          Rakesh R added a comment -

          Attached patch addressing Sean Busbey comments.

          Show
          Rakesh R added a comment - Attached patch addressing Sean Busbey comments.
          Hide
          Sean Busbey added a comment -

          the patch look good. have a draft of the needed release note?

          Show
          Sean Busbey added a comment - the patch look good. have a draft of the needed release note?
          Hide
          Haohui Mai added a comment -
          -  public static final Log LOG = LogFactory.getLog(CachePoolInfo.class);
          +  public static final Logger LOG = LoggerFactory
          +      .getLogger(CachePoolInfo.class);
          

          The LOG variable is never used. And in terms of compatibility this should be fine from a practical point of view. There are no changes on the members that actually hold the data.

          Show
          Haohui Mai added a comment - - public static final Log LOG = LogFactory.getLog(CachePoolInfo.class); + public static final Logger LOG = LoggerFactory + .getLogger(CachePoolInfo.class); The LOG variable is never used. And in terms of compatibility this should be fine from a practical point of view. There are no changes on the members that actually hold the data.
          Hide
          Hadoop QA added a comment -



          -1 overall



          Vote Subsystem Runtime Comment
          0 pre-patch 15m 36s Pre-patch trunk compilation is healthy.
          +1 @author 0m 0s The patch does not contain any @author tags.
          -1 tests included 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
          +1 javac 7m 32s There were no new javac warning messages.
          +1 javadoc 9m 42s There were no new javadoc warning messages.
          +1 release audit 0m 23s The applied patch does not increase the total number of release audit warnings.
          +1 checkstyle 0m 21s There were no new checkstyle issues.
          +1 whitespace 0m 1s The patch has no lines that end in whitespace.
          +1 install 1m 37s mvn install still works.
          +1 eclipse:eclipse 0m 34s The patch built with eclipse:eclipse.
          +1 findbugs 0m 52s The patch does not introduce any new Findbugs (version 3.0.0) warnings.
          +1 native 3m 16s Pre-build of native portion
          +1 hdfs tests 0m 16s Tests passed in hadoop-hdfs-client.
              40m 14s  



          Subsystem Report/Notes
          Patch URL http://issues.apache.org/jira/secure/attachment/12740186/HDFS-6564-02.patch
          Optional Tests javadoc javac unit findbugs checkstyle
          git revision trunk / 015535d
          hadoop-hdfs-client test log https://builds.apache.org/job/PreCommit-HDFS-Build/11394/artifact/patchprocess/testrun_hadoop-hdfs-client.txt
          Test Results https://builds.apache.org/job/PreCommit-HDFS-Build/11394/testReport/
          Java 1.7.0_55
          uname Linux asf900.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
          Console output https://builds.apache.org/job/PreCommit-HDFS-Build/11394/console

          This message was automatically generated.

          Show
          Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 pre-patch 15m 36s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. -1 tests included 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. +1 javac 7m 32s There were no new javac warning messages. +1 javadoc 9m 42s There were no new javadoc warning messages. +1 release audit 0m 23s The applied patch does not increase the total number of release audit warnings. +1 checkstyle 0m 21s There were no new checkstyle issues. +1 whitespace 0m 1s The patch has no lines that end in whitespace. +1 install 1m 37s mvn install still works. +1 eclipse:eclipse 0m 34s The patch built with eclipse:eclipse. +1 findbugs 0m 52s The patch does not introduce any new Findbugs (version 3.0.0) warnings. +1 native 3m 16s Pre-build of native portion +1 hdfs tests 0m 16s Tests passed in hadoop-hdfs-client.     40m 14s   Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12740186/HDFS-6564-02.patch Optional Tests javadoc javac unit findbugs checkstyle git revision trunk / 015535d hadoop-hdfs-client test log https://builds.apache.org/job/PreCommit-HDFS-Build/11394/artifact/patchprocess/testrun_hadoop-hdfs-client.txt Test Results https://builds.apache.org/job/PreCommit-HDFS-Build/11394/testReport/ Java 1.7.0_55 uname Linux asf900.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Console output https://builds.apache.org/job/PreCommit-HDFS-Build/11394/console This message was automatically generated.
          Hide
          Rakesh R added a comment -

          Thanks Haohui Mai for pointing out this, yes its never used. But I could see CachePoolInfo is marked as public interface, do we need to retain the LOG considering this?

          @InterfaceAudience.Public
          @InterfaceStability.Evolving
          public class CachePoolInfo {
          
          Show
          Rakesh R added a comment - Thanks Haohui Mai for pointing out this, yes its never used. But I could see CachePoolInfo is marked as public interface, do we need to retain the LOG considering this? @InterfaceAudience.Public @InterfaceStability.Evolving public class CachePoolInfo {
          Hide
          Haohui Mai added a comment -

          I think it is okay to get rid of the LOG member since it is a evolving interface. According to http://hadoop.apache.org/docs/current/hadoop-project-dist/hadoop-common/InterfaceClassification.html:

          Evolving
          Evolving, but incompatible changes are allowed at minor release (i.e. m .x)

          Show
          Haohui Mai added a comment - I think it is okay to get rid of the LOG member since it is a evolving interface. According to http://hadoop.apache.org/docs/current/hadoop-project-dist/hadoop-common/InterfaceClassification.html: Evolving Evolving, but incompatible changes are allowed at minor release (i.e. m .x)
          Hide
          Sean Busbey added a comment -

          Whatever you decide to do about the LOG member, if it involves the class changing or it being removed please remember to flag the change and make a release note. That way downstream folks find out about it up front rather than by surprise when they are broken.

          Show
          Sean Busbey added a comment - Whatever you decide to do about the LOG member, if it involves the class changing or it being removed please remember to flag the change and make a release note. That way downstream folks find out about it up front rather than by surprise when they are broken.
          Rakesh R made changes -
          Attachment HDFS-6564-03.patch [ 12740553 ]
          Hide
          Rakesh R added a comment -

          Thanks Sean Busbey. Below is the draft release note, any comments. If everyone agree with this then will update the jira.

          Users may need attention for this change while upgrading to this version. org.apache.hadoop.hdfs.protocol.CachePoolInfo#LOG static member variable has been removed as it is not used anywhere. Users need to correct their code if any one has a reference to this variable.
          
          Show
          Rakesh R added a comment - Thanks Sean Busbey . Below is the draft release note, any comments. If everyone agree with this then will update the jira. Users may need attention for this change while upgrading to this version. org.apache.hadoop.hdfs.protocol.CachePoolInfo#LOG static member variable has been removed as it is not used anywhere. Users need to correct their code if any one has a reference to this variable.
          Hide
          Rakesh R added a comment -

          Thanks Haohui Mai for the advice. Attached another patch by removing the LOG variable.

          Show
          Rakesh R added a comment - Thanks Haohui Mai for the advice. Attached another patch by removing the LOG variable.
          Hide
          Sean Busbey added a comment -

          You should include mention of how to get the same functionality. In this case, they can retrieve the named logger via the logging framework of their choice directly. Since the previous member was commons-logging object, that would be done via LogFactory.getLog(CachePoolInfo.class) or LogFactory.getLog("org.apache.hadoop.hdfs.protocol.CachePoolInfo").

          Show
          Sean Busbey added a comment - You should include mention of how to get the same functionality. In this case, they can retrieve the named logger via the logging framework of their choice directly. Since the previous member was commons-logging object, that would be done via LogFactory.getLog(CachePoolInfo.class) or LogFactory.getLog("org.apache.hadoop.hdfs.protocol.CachePoolInfo") .
          Hide
          Rakesh R added a comment -

          I've modified the note, kindly review it again.

          Users may need attention for this change while upgrading to this version. org.apache.hadoop.hdfs.protocol.CachePoolInfo#LOG public static member variable has been removed as it is not used anywhere. Users need to correct their code if any one has a reference to this variable. One can retrieve the named logger via the logging framework of their choice directly like, org.apache.commons.logging.Log LOG = org.apache.commons.logging.LogFactory.getLogger(org.apache.hadoop.hdfs.protocol.CachePoolInfo.class);
          
          Show
          Rakesh R added a comment - I've modified the note, kindly review it again. Users may need attention for this change while upgrading to this version. org.apache.hadoop.hdfs.protocol.CachePoolInfo#LOG public static member variable has been removed as it is not used anywhere. Users need to correct their code if any one has a reference to this variable. One can retrieve the named logger via the logging framework of their choice directly like, org.apache.commons.logging.Log LOG = org.apache.commons.logging.LogFactory.getLogger(org.apache.hadoop.hdfs.protocol.CachePoolInfo.class);
          Hide
          Rakesh R added a comment -

          Please ignore the previous, small correction to the logger class.

          Users may need attention for this change while upgrading to this version. org.apache.hadoop.hdfs.protocol.CachePoolInfo#LOG public static member variable has been removed as it is not used anywhere. Users need to correct their code if any one has a reference to this variable. One can retrieve the named logger via the logging framework of their choice directly like, org.slf4j.Logger LOG = org.slf4j.LoggerFactory.getLogger(org.apache.hadoop.hdfs.protocol.CachePoolInfo.class);
          
          Show
          Rakesh R added a comment - Please ignore the previous, small correction to the logger class. Users may need attention for this change while upgrading to this version. org.apache.hadoop.hdfs.protocol.CachePoolInfo#LOG public static member variable has been removed as it is not used anywhere. Users need to correct their code if any one has a reference to this variable. One can retrieve the named logger via the logging framework of their choice directly like, org.slf4j.Logger LOG = org.slf4j.LoggerFactory.getLogger(org.apache.hadoop.hdfs.protocol.CachePoolInfo.class);
          Hide
          Sean Busbey added a comment -

          Since before removal the LOG was commons.logging, I would like to include how to get the member from that framework. Including how to do it in slf4j as well sounds fine.

          Show
          Sean Busbey added a comment - Since before removal the LOG was commons.logging, I would like to include how to get the member from that framework. Including how to do it in slf4j as well sounds fine.
          Hide
          Rakesh R added a comment -

          We have an example mentioned in the previous note.

          org.slf4j.Logger LOG = org.slf4j.LoggerFactory.getLogger(org.apache.hadoop.hdfs.protocol.CachePoolInfo.class);
          

          I think I can include link of slf4j manual in the note like below. Does this sound good to you ?

          Users may need special attention for this change while upgrading to this version. Previously hdfs client was using commons-logging as the logging framework. With this change it uses slf4j framework. For more details: http://www.slf4j.org/manual.html. Also, org.apache.hadoop.hdfs.protocol.CachePoolInfo#LOG public static member variable has been removed as it is not used anywhere. Users need to correct their code if any one has a reference to this variable. One can retrieve the named logger via the logging framework of their choice directly like, org.apache.commons.logging.Log LOG = org.apache.commons.logging.LogFactory.getLogger(org.apache.hadoop.hdfs.protocol.CachePoolInfo.class);
          
          Show
          Rakesh R added a comment - We have an example mentioned in the previous note. org.slf4j.Logger LOG = org.slf4j.LoggerFactory.getLogger(org.apache.hadoop.hdfs.protocol.CachePoolInfo.class); I think I can include link of slf4j manual in the note like below. Does this sound good to you ? Users may need special attention for this change while upgrading to this version. Previously hdfs client was using commons-logging as the logging framework. With this change it uses slf4j framework. For more details: http: //www.slf4j.org/manual.html. Also, org.apache.hadoop.hdfs.protocol.CachePoolInfo#LOG public static member variable has been removed as it is not used anywhere. Users need to correct their code if any one has a reference to this variable. One can retrieve the named logger via the logging framework of their choice directly like, org.apache.commons.logging.Log LOG = org.apache.commons.logging.LogFactory.getLogger(org.apache.hadoop.hdfs.protocol.CachePoolInfo.class);
          Hide
          Rakesh R added a comment -

          We have an example mentioned in the previous note, is that fine?

          org.slf4j.Logger LOG = org.slf4j.LoggerFactory.getLogger(org.apache.hadoop.hdfs.protocol.CachePoolInfo.class);
          

          I think I can include link of slf4j manual in the note like, For more details: http://www.slf4j.org/manual.html. Does this sound good to you ?

          Users may need special attention for this change while upgrading to this version. Previously hdfs client was using commons-logging as the logging framework. With this change it uses slf4j framework. For more details: http://www.slf4j.org/manual.html. Also, org.apache.hadoop.hdfs.protocol.CachePoolInfo#LOG public static member variable has been removed as it is not used anywhere. Users need to correct their code if any one has a reference to this variable. One can retrieve the named logger via the logging framework of their choice directly like, org.slf4j.Logger LOG = org.slf4j.LoggerFactory.getLogger((org.apache.hadoop.hdfs.protocol.CachePoolInfo.class);
          
          Show
          Rakesh R added a comment - We have an example mentioned in the previous note, is that fine? org.slf4j.Logger LOG = org.slf4j.LoggerFactory.getLogger(org.apache.hadoop.hdfs.protocol.CachePoolInfo.class); I think I can include link of slf4j manual in the note like, For more details: http://www.slf4j.org/manual.html . Does this sound good to you ? Users may need special attention for this change while upgrading to this version. Previously hdfs client was using commons-logging as the logging framework. With this change it uses slf4j framework. For more details: http: //www.slf4j.org/manual.html. Also, org.apache.hadoop.hdfs.protocol.CachePoolInfo#LOG public static member variable has been removed as it is not used anywhere. Users need to correct their code if any one has a reference to this variable. One can retrieve the named logger via the logging framework of their choice directly like, org.slf4j.Logger LOG = org.slf4j.LoggerFactory.getLogger((org.apache.hadoop.hdfs.protocol.CachePoolInfo.class);
          Hide
          Hadoop QA added a comment -



          -1 overall



          Vote Subsystem Runtime Comment
          0 pre-patch 15m 16s Pre-patch trunk compilation is healthy.
          +1 @author 0m 0s The patch does not contain any @author tags.
          -1 tests included 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
          +1 javac 7m 28s There were no new javac warning messages.
          +1 javadoc 9m 34s There were no new javadoc warning messages.
          +1 release audit 0m 22s The applied patch does not increase the total number of release audit warnings.
          +1 checkstyle 0m 21s There were no new checkstyle issues.
          +1 whitespace 0m 0s The patch has no lines that end in whitespace.
          +1 install 1m 32s mvn install still works.
          +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse.
          +1 findbugs 0m 51s The patch does not introduce any new Findbugs (version 3.0.0) warnings.
          +1 native 3m 13s Pre-build of native portion
          +1 hdfs tests 0m 16s Tests passed in hadoop-hdfs-client.
              39m 30s  



          Subsystem Report/Notes
          Patch URL http://issues.apache.org/jira/secure/attachment/12740553/HDFS-6564-03.patch
          Optional Tests javadoc javac unit findbugs checkstyle
          git revision trunk / 49f5d20
          hadoop-hdfs-client test log https://builds.apache.org/job/PreCommit-HDFS-Build/11416/artifact/patchprocess/testrun_hadoop-hdfs-client.txt
          Test Results https://builds.apache.org/job/PreCommit-HDFS-Build/11416/testReport/
          Java 1.7.0_55
          uname Linux asf902.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
          Console output https://builds.apache.org/job/PreCommit-HDFS-Build/11416/console

          This message was automatically generated.

          Show
          Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 pre-patch 15m 16s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. -1 tests included 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. +1 javac 7m 28s There were no new javac warning messages. +1 javadoc 9m 34s There were no new javadoc warning messages. +1 release audit 0m 22s The applied patch does not increase the total number of release audit warnings. +1 checkstyle 0m 21s There were no new checkstyle issues. +1 whitespace 0m 0s The patch has no lines that end in whitespace. +1 install 1m 32s mvn install still works. +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse. +1 findbugs 0m 51s The patch does not introduce any new Findbugs (version 3.0.0) warnings. +1 native 3m 13s Pre-build of native portion +1 hdfs tests 0m 16s Tests passed in hadoop-hdfs-client.     39m 30s   Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12740553/HDFS-6564-03.patch Optional Tests javadoc javac unit findbugs checkstyle git revision trunk / 49f5d20 hadoop-hdfs-client test log https://builds.apache.org/job/PreCommit-HDFS-Build/11416/artifact/patchprocess/testrun_hadoop-hdfs-client.txt Test Results https://builds.apache.org/job/PreCommit-HDFS-Build/11416/testReport/ Java 1.7.0_55 uname Linux asf902.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Console output https://builds.apache.org/job/PreCommit-HDFS-Build/11416/console This message was automatically generated.
          Rakesh R made changes -
          Release Note Users may need special attention for this change while upgrading to this version. Previously hdfs client was using commons-logging as the logging framework. With this change it will use slf4j framework. For more details about slf4j, please see: http://www.slf4j.org/manual.html. Also, org.apache.hadoop.hdfs.protocol.CachePoolInfo#LOG public static member variable has been removed as it is not used anywhere. Users need to correct their code if any one has a reference to this variable. One can retrieve the named logger via the logging framework of their choice directly like, org.slf4j.Logger LOG = org.slf4j.LoggerFactory.getLogger(org.apache.hadoop.hdfs.protocol.CachePoolInfo.class);
          Hide
          Rakesh R added a comment -

          Thanks for the reviews. I've updated Release note section in the jira. Anything else required for this change.

          Show
          Rakesh R added a comment - Thanks for the reviews. I've updated Release note section in the jira. Anything else required for this change.
          Hide
          Haohui Mai added a comment -

          +1. I'll commit it shortly.

          Show
          Haohui Mai added a comment - +1. I'll commit it shortly.
          Hide
          Haohui Mai added a comment -

          I've committed the patch to trunk and branch-2. Thanks Rakesh R for the contribution.

          Show
          Haohui Mai added a comment - I've committed the patch to trunk and branch-2. Thanks Rakesh R for the contribution.
          Haohui Mai made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Hadoop Flags Reviewed [ 10343 ]
          Fix Version/s 2.8.0 [ 12329057 ]
          Resolution Fixed [ 1 ]
          Hide
          Rakesh R added a comment -

          Thanks Haohui Mai for the review and commit. Thanks Sean Busbey for the review.

          Show
          Rakesh R added a comment - Thanks Haohui Mai for the review and commit. Thanks Sean Busbey for the review.
          Hide
          Hudson added a comment -

          FAILURE: Integrated in Hadoop-Yarn-trunk #968 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/968/)
          HDFS-6564. Use slf4j instead of common-logging in hdfs-client. Contributed by Rakesh R. (wheat9: rev 122cad6aec5839d8d515c5008425ecb34f2fa56b)

          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/TokenAspect.java
          • hadoop-hdfs-project/hadoop-hdfs-client/pom.xml
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/BlockStoragePolicy.java
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/CachePoolInfo.java
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/URLConnectionFactory.java
          • hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DFSUtilClient.java
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/WebHdfsFileSystem.java
          Show
          Hudson added a comment - FAILURE: Integrated in Hadoop-Yarn-trunk #968 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/968/ ) HDFS-6564 . Use slf4j instead of common-logging in hdfs-client. Contributed by Rakesh R. (wheat9: rev 122cad6aec5839d8d515c5008425ecb34f2fa56b) hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/TokenAspect.java hadoop-hdfs-project/hadoop-hdfs-client/pom.xml hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/BlockStoragePolicy.java hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/CachePoolInfo.java hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/URLConnectionFactory.java hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DFSUtilClient.java hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/WebHdfsFileSystem.java
          Hide
          Hudson added a comment -

          SUCCESS: Integrated in Hadoop-Yarn-trunk-Java8 #238 (See https://builds.apache.org/job/Hadoop-Yarn-trunk-Java8/238/)
          HDFS-6564. Use slf4j instead of common-logging in hdfs-client. Contributed by Rakesh R. (wheat9: rev 122cad6aec5839d8d515c5008425ecb34f2fa56b)

          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/BlockStoragePolicy.java
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/WebHdfsFileSystem.java
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DFSUtilClient.java
          • hadoop-hdfs-project/hadoop-hdfs-client/pom.xml
          • hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/CachePoolInfo.java
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/TokenAspect.java
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/URLConnectionFactory.java
          Show
          Hudson added a comment - SUCCESS: Integrated in Hadoop-Yarn-trunk-Java8 #238 (See https://builds.apache.org/job/Hadoop-Yarn-trunk-Java8/238/ ) HDFS-6564 . Use slf4j instead of common-logging in hdfs-client. Contributed by Rakesh R. (wheat9: rev 122cad6aec5839d8d515c5008425ecb34f2fa56b) hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/BlockStoragePolicy.java hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/WebHdfsFileSystem.java hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DFSUtilClient.java hadoop-hdfs-project/hadoop-hdfs-client/pom.xml hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/CachePoolInfo.java hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/TokenAspect.java hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/URLConnectionFactory.java
          Hide
          Hudson added a comment -

          FAILURE: Integrated in Hadoop-Hdfs-trunk-Java8 #227 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Java8/227/)
          HDFS-6564. Use slf4j instead of common-logging in hdfs-client. Contributed by Rakesh R. (wheat9: rev 122cad6aec5839d8d515c5008425ecb34f2fa56b)

          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/WebHdfsFileSystem.java
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/BlockStoragePolicy.java
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/URLConnectionFactory.java
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/CachePoolInfo.java
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/TokenAspect.java
          • hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DFSUtilClient.java
          • hadoop-hdfs-project/hadoop-hdfs-client/pom.xml
          Show
          Hudson added a comment - FAILURE: Integrated in Hadoop-Hdfs-trunk-Java8 #227 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Java8/227/ ) HDFS-6564 . Use slf4j instead of common-logging in hdfs-client. Contributed by Rakesh R. (wheat9: rev 122cad6aec5839d8d515c5008425ecb34f2fa56b) hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/WebHdfsFileSystem.java hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/BlockStoragePolicy.java hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/URLConnectionFactory.java hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/CachePoolInfo.java hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/TokenAspect.java hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DFSUtilClient.java hadoop-hdfs-project/hadoop-hdfs-client/pom.xml
          Hide
          Hudson added a comment -

          SUCCESS: Integrated in Hadoop-Hdfs-trunk #2166 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/2166/)
          HDFS-6564. Use slf4j instead of common-logging in hdfs-client. Contributed by Rakesh R. (wheat9: rev 122cad6aec5839d8d515c5008425ecb34f2fa56b)

          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/CachePoolInfo.java
          • hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/TokenAspect.java
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/BlockStoragePolicy.java
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/URLConnectionFactory.java
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/WebHdfsFileSystem.java
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DFSUtilClient.java
          • hadoop-hdfs-project/hadoop-hdfs-client/pom.xml
          Show
          Hudson added a comment - SUCCESS: Integrated in Hadoop-Hdfs-trunk #2166 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/2166/ ) HDFS-6564 . Use slf4j instead of common-logging in hdfs-client. Contributed by Rakesh R. (wheat9: rev 122cad6aec5839d8d515c5008425ecb34f2fa56b) hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/CachePoolInfo.java hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/TokenAspect.java hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/BlockStoragePolicy.java hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/URLConnectionFactory.java hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/WebHdfsFileSystem.java hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DFSUtilClient.java hadoop-hdfs-project/hadoop-hdfs-client/pom.xml
          Hide
          Hudson added a comment -

          FAILURE: Integrated in Hadoop-Mapreduce-trunk-Java8 #236 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Java8/236/)
          HDFS-6564. Use slf4j instead of common-logging in hdfs-client. Contributed by Rakesh R. (wheat9: rev 122cad6aec5839d8d515c5008425ecb34f2fa56b)

          • hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/CachePoolInfo.java
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/BlockStoragePolicy.java
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DFSUtilClient.java
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/URLConnectionFactory.java
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/TokenAspect.java
          • hadoop-hdfs-project/hadoop-hdfs-client/pom.xml
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/WebHdfsFileSystem.java
          Show
          Hudson added a comment - FAILURE: Integrated in Hadoop-Mapreduce-trunk-Java8 #236 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Java8/236/ ) HDFS-6564 . Use slf4j instead of common-logging in hdfs-client. Contributed by Rakesh R. (wheat9: rev 122cad6aec5839d8d515c5008425ecb34f2fa56b) hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/CachePoolInfo.java hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/BlockStoragePolicy.java hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DFSUtilClient.java hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/URLConnectionFactory.java hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/TokenAspect.java hadoop-hdfs-project/hadoop-hdfs-client/pom.xml hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/WebHdfsFileSystem.java
          Hide
          Hudson added a comment -

          FAILURE: Integrated in Hadoop-Mapreduce-trunk #2184 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/2184/)
          HDFS-6564. Use slf4j instead of common-logging in hdfs-client. Contributed by Rakesh R. (wheat9: rev 122cad6aec5839d8d515c5008425ecb34f2fa56b)

          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DFSUtilClient.java
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/CachePoolInfo.java
          • hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/TokenAspect.java
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/WebHdfsFileSystem.java
          • hadoop-hdfs-project/hadoop-hdfs-client/pom.xml
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/URLConnectionFactory.java
          • hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/BlockStoragePolicy.java
          Show
          Hudson added a comment - FAILURE: Integrated in Hadoop-Mapreduce-trunk #2184 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/2184/ ) HDFS-6564 . Use slf4j instead of common-logging in hdfs-client. Contributed by Rakesh R. (wheat9: rev 122cad6aec5839d8d515c5008425ecb34f2fa56b) hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DFSUtilClient.java hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/CachePoolInfo.java hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/TokenAspect.java hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/WebHdfsFileSystem.java hadoop-hdfs-project/hadoop-hdfs-client/pom.xml hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/URLConnectionFactory.java hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/BlockStoragePolicy.java
          Allen Wittenauer made changes -
          Hadoop Flags Reviewed [ 10343 ] Incompatible change,Reviewed [ 10342, 10343 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Patch Available Patch Available
          363d 10h 56m 1 Rakesh R 17/Jun/15 12:35
          Patch Available Patch Available Resolved Resolved
          6d 7h 6m 1 Haohui Mai 23/Jun/15 19:41

            People

            • Assignee:
              Rakesh R
              Reporter:
              Haohui Mai
            • Votes:
              0 Vote for this issue
              Watchers:
              8 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development