[ 
https://issues.apache.org/jira/browse/HBASE-5041?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13175247#comment-13175247
 ] 

ramkrishna.s.vasudevan commented on HBASE-5041:
-----------------------------------------------

@Shrijeet
{code}
if (isRegionName(tableNameOrRegionName)) {
        Pair<HRegionInfo, HServerAddress> pair =
          MetaReader.getRegion(ct, tableNameOrRegionName);
        if (pair == null || pair.getSecond() == null) {
          LOG.info("No server in .META. for " +
            Bytes.toStringBinary(tableNameOrRegionName) + "; pair=" + pair);
{code}
The MetaReader.getRegion should return null pair if a wrong table name is 
provided.  Can't we throw exception there? May be am missing something.
                
> Major compaction on non existing table does not throw error 
> ------------------------------------------------------------
>
>                 Key: HBASE-5041
>                 URL: https://issues.apache.org/jira/browse/HBASE-5041
>             Project: HBase
>          Issue Type: Bug
>          Components: regionserver, shell
>    Affects Versions: 0.90.3
>            Reporter: Shrijeet Paliwal
>            Assignee: Shrijeet Paliwal
>             Fix For: 0.92.0, 0.94.0, 0.90.6
>
>         Attachments: 0001-HBASE-5041-Throw-error-if-table-does-not-exist.patch
>
>
> Following will not complain even if fubar does not exist
> {code}
> echo "major_compact 'fubar'" | $HBASE_HOME/bin/hbase shell
> {code}
> The downside for this defect is that major compaction may be skipped due to
> a typo by Ops.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to