You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "Sean Busbey (JIRA)" <ji...@apache.org> on 2018/04/03 14:16:00 UTC

[jira] [Updated] (HBASE-20253) Error message is missing for restore_snapshot

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

Sean Busbey updated HBASE-20253:
--------------------------------
    Status: In Progress  (was: Patch Available)

I believe we still need to re-throw the exception even with the improved message. Otherwise, the shell will claim that the operation succeeded because the {{puts}} succeeds and is the last operation.

example prior to this patch:

{code}
$ hbase shell --noninteractive <<EOF
> create 'test:some_table', 'family'
> snapshot 'test:some_table', 'example_snapshot'
> restore_snapshot 'example_snapshot'
> EOF
Created table test:some_table
Took 1.1634 seconds                                                                                      
Hbase::Table - test:some_table
Took 0.5606 seconds                                                                                      

Took 0.1246 seconds                                                                                      java exception
ERROR Java::OrgApacheHadoopHbase::TableNotDisabledException: test:some_table
[root@hbase19158-1 ~]# echo $?
1
{code}

example with this patch:

{code}
$ hbase shell --noninteractive <<EOF
> create 'test:some_table', 'family'
> snapshot 'test:some_table', 'example_snapshot'
> restore_snapshot 'example_snapshot'
> EOF
Created table test:some_table
Took 1.1208 seconds                                                                                      
Hbase::Table - test:some_table
Took 0.4879 seconds                                                                                      

Table 'test:some_table' should be disabled to restore snapshot.
Took 0.1080 seconds                                                                                      

[root@hbase19158-1 ~]# echo $?
0
{code}

It'd be good to add a test that checks this negative example while we're here.

> Error message is missing for restore_snapshot
> ---------------------------------------------
>
>                 Key: HBASE-20253
>                 URL: https://issues.apache.org/jira/browse/HBASE-20253
>             Project: HBase
>          Issue Type: Sub-task
>          Components: shell
>    Affects Versions: 2.0.0
>            Reporter: Peter Somogyi
>            Assignee: Gabor Bota
>            Priority: Minor
>         Attachments: HBASE-20253.master.001.patch, HBASE-20253.master.002.patch
>
>
> When the table is not disabled and restore_snapshot executed the error message is useless, only displays the table name.
> hbase(main):007:0> restore_snapshot 'tsnap'
> ERROR: t
> Restore a specified snapshot.
> The restore will replace the content of the original table,
> bringing back the content to the snapshot state.
> The table must be disabled.
> Examples:
>   hbase> restore_snapshot 'snapshotName'
> Following command will restore all acl from snapshot table into the table.
>   hbase> restore_snapshot 'snapshotName', \{RESTORE_ACL=>true}
> Took 0.1044 seconds



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)