You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hbase.apache.org by "Eric Tschetter (JIRA)" <ji...@apache.org> on 2009/09/25 00:10:15 UTC

[jira] Updated: (HBASE-1865) 0.20.0 TableInputFormatBase NPE

     [ https://issues.apache.org/jira/browse/HBASE-1865?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Eric Tschetter updated HBASE-1865:
----------------------------------

    Description: 
Spot the bug in this code:

public List<InputSplit> getSplits(JobContext context) throws IOException {
    byte [][] startKeys = table.getStartKeys();
    if (startKeys == null || startKeys.length == 0) {
      throw new IOException("Expecting at least one region.");
    }
    if (table == null) {
      throw new IOException("No table was provided.");
    }
...
}

Should check if the table is null before calling a method on it.

Admittedly, this isn't the worst bug in the world, it's really just more of a nuisance in that the "No table was provided" message becomes an NPE

  was:
Spot the bug in this code:

public List<InputSplit> getSplits(JobContext context) throws IOException {
    byte [][] startKeys = table.getStartKeys();
    if (startKeys == null || startKeys.length == 0) {
      throw new IOException("Expecting at least one region.");
    }
    if (table == null) {
      throw new IOException("No table was provided.");
    }
...
}

Should check if the table is null before calling a method on it.




> 0.20.0 TableInputFormatBase NPE
> -------------------------------
>
>                 Key: HBASE-1865
>                 URL: https://issues.apache.org/jira/browse/HBASE-1865
>             Project: Hadoop HBase
>          Issue Type: Bug
>    Affects Versions: 0.20.0
>            Reporter: Eric Tschetter
>
> Spot the bug in this code:
> public List<InputSplit> getSplits(JobContext context) throws IOException {
>     byte [][] startKeys = table.getStartKeys();
>     if (startKeys == null || startKeys.length == 0) {
>       throw new IOException("Expecting at least one region.");
>     }
>     if (table == null) {
>       throw new IOException("No table was provided.");
>     }
> ...
> }
> Should check if the table is null before calling a method on it.
> Admittedly, this isn't the worst bug in the world, it's really just more of a nuisance in that the "No table was provided" message becomes an NPE

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.