Review Board 1.7.22


HBASE-6777: Snapshot Restore Interface

Review Request #7096 - Created Sept. 13, 2012 and submitted

Matteo Bertozzi
trunk
HBASE-6777
Reviewers
hbase
jmhsieh, jyates
hbase-git
Restore Snapshot Interface on the same line as "Take a snapshot" HBASE-6765

 
Total:
26
Open:
6
Resolved:
19
Dropped:
1
Status:
From:
Description From Last Updated Status
I don't see this exception. Ted Yu Sept. 14, 2012, 6:03 a.m. Open
nit: formatted incorrectly how? Jesse Yates Sept. 25, 2012, 12:25 a.m. Open
nit: comment about how you can't restore multiple tables at the same time? In the impl we need to remove ... Jesse Yates Sept. 25, 2012, 12:25 a.m. Open
This looks really similar to the take a snapshot stuff - maybe wrap it all in something like a SnapshotCallable? Jesse Yates Sept. 25, 2012, 12:25 a.m. Open
Might want to consider explicitly calling out the fact that the table will be offlined to roll it back to ... Jesse Yates Nov. 1, 2012, 10:26 p.m. Open
what do we want to do with the first snapshot if we are successful? should we delete it? Maybe add ... Jesse Yates Nov. 15, 2012, 10:22 p.m. Open
Review request changed
Updated (Jan. 8, 2013, 5:32 p.m.)
  • changed from pending to submitted