Uma Maheswara Rao G commented on HBASE-5645:

1) ShutdownHook.java
 if (cache == null) {
        throw new RuntimeException(
            "This should not happen. Could not find the cache class in 

      Field field = null;
      try {
        field = cache.getDeclaredField(CLIENT_FINALIZER_DATA_METHOD);
      } catch (NoSuchFieldException e) {
        // We can get here if the Cache class does not have a clientFinalizer
        // instance: i.e. we're running on straight 0.20 w/o hadoop-4829.

2) In Store.java
 byte[] firstKey = reader.getFirstRowKey();
      Preconditions.checkState(firstKey != null, "First key can not be null");
      byte[] lk = reader.getLastKey();
      Preconditions.checkState(lk != null, "Last key can not be null");
      byte[] lastKey =  KeyValue.createKeyValueFromKey(lk).getRow();

We already handled this with preconditions.

3) {code}
  int logCount = this.outputfiles == null? 0: this.outputfiles.size();
+    int logCount = this.outputfiles.size();
This change also already handled by Hbase-5653. So may need to update with 
Mostly all are need to resolve the conflicts with fixed and committed changes.
> [findbugs] Fix correctness warnings
> -----------------------------------
>                 Key: HBASE-5645
>                 URL: https://issues.apache.org/jira/browse/HBASE-5645
>             Project: HBase
>          Issue Type: Sub-task
>          Components: scripts
>            Reporter: Jonathan Hsieh
>            Assignee: David S. Wang
>         Attachments: HBASE-5645-2.patch, HBASE-5645.patch, HBASE-5645.patch
> See 
> https://builds.apache.org/job/PreCommit-HBASE-Build/1313//artifact/trunk/patchprocess/newPatchFindbugsWarnings.html
> Fix the warnings in the correctness section.

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
For more information on JIRA, see: http://www.atlassian.com/software/jira


Reply via email to