[jira] [Commented] (HBASE-16520) TableBackupProcedure ctor should set procedure owner

2016-08-30 Thread Ted Yu (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16520?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15450019#comment-15450019 ] Ted Yu commented on HBASE-16520: Mind continuing comment over HBASE-16532 ? Thanks >

[jira] [Commented] (HBASE-16520) TableBackupProcedure ctor should set procedure owner

2016-08-30 Thread Ted Yu (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16520?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15449982#comment-15449982 ] Ted Yu commented on HBASE-16520: Is there API for obtaining hbase module given class ? What if new

[jira] [Commented] (HBASE-16520) TableBackupProcedure ctor should set procedure owner

2016-08-30 Thread Matteo Bertozzi (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16520?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15449957#comment-15449957 ] Matteo Bertozzi commented on HBASE-16520: - by trying to start the cluster and execute the

[jira] [Commented] (HBASE-16520) TableBackupProcedure ctor should set procedure owner

2016-08-30 Thread Ted Yu (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16520?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15449950#comment-15449950 ] Ted Yu commented on HBASE-16520: If "check owner set" is turned off for tests, how would we find the

[jira] [Commented] (HBASE-16520) TableBackupProcedure ctor should set procedure owner

2016-08-30 Thread Matteo Bertozzi (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16520?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15449773#comment-15449773 ] Matteo Bertozzi commented on HBASE-16520: - I'd like this idea of forcing the owner. It should be

[jira] [Commented] (HBASE-16520) TableBackupProcedure ctor should set procedure owner

2016-08-30 Thread Ted Yu (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16520?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15449691#comment-15449691 ] Ted Yu commented on HBASE-16520: Another approach is to call proc.getOwner() when proc is loaded first

[jira] [Commented] (HBASE-16520) TableBackupProcedure ctor should set procedure owner

2016-08-30 Thread Ted Yu (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16520?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15449660#comment-15449660 ] Ted Yu commented on HBASE-16520: isProcedureOwner() doesn't declare to throw IOE. It would be a surprise

[jira] [Commented] (HBASE-16520) TableBackupProcedure ctor should set procedure owner

2016-08-30 Thread Matteo Bertozzi (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16520?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15449595#comment-15449595 ] Matteo Bertozzi commented on HBASE-16520: - having an owner at least from non-testing procedure is

[jira] [Commented] (HBASE-16520) TableBackupProcedure ctor should set procedure owner

2016-08-30 Thread Ted Yu (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-16520?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15449572#comment-15449572 ] Ted Yu commented on HBASE-16520: [~mbertozzi]: ProcedureExecutor#isProcedureOwner() assumes every