[jira] [Commented] (HBASE-17123) Add postBulkLoadHFile variant that notifies the final paths for the hfiles

2017-12-04 Thread Anoop Sam John (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-17123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16278009#comment-16278009 ] Anoop Sam John commented on HBASE-17123: CPs backward compat we can break IMO. We broke it left

[jira] [Commented] (HBASE-17123) Add postBulkLoadHFile variant that notifies the final paths for the hfiles

2017-12-04 Thread Appy (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-17123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16277367#comment-16277367 ] Appy commented on HBASE-17123: -- you're right, bulk loading is over, so maybe false cannot be interpreted as

[jira] [Commented] (HBASE-17123) Add postBulkLoadHFile variant that notifies the final paths for the hfiles

2017-12-04 Thread Anoop Sam John (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-17123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16277123#comment-16277123 ] Anoop Sam John commented on HBASE-17123: {code} if (region.getCoprocessorHost() != null) {

[jira] [Commented] (HBASE-17123) Add postBulkLoadHFile variant that notifies the final paths for the hfiles

2017-12-04 Thread Anoop Sam John (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-17123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16277121#comment-16277121 ] Anoop Sam John commented on HBASE-17123: But the bulk loading is over by this time the post hook

[jira] [Commented] (HBASE-17123) Add postBulkLoadHFile variant that notifies the final paths for the hfiles

2017-12-04 Thread Appy (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-17123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16277090#comment-16277090 ] Appy commented on HBASE-17123: -- Earlier, it allowed CP to fail a bulkload by returning false. Our

[jira] [Commented] (HBASE-17123) Add postBulkLoadHFile variant that notifies the final paths for the hfiles

2017-12-04 Thread Anoop Sam John (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-17123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16276680#comment-16276680 ] Anoop Sam John commented on HBASE-17123: {code} if (region.getCoprocessorHost() != null) {

[jira] [Commented] (HBASE-17123) Add postBulkLoadHFile variant that notifies the final paths for the hfiles

2017-12-01 Thread Appy (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-17123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16275097#comment-16275097 ] Appy commented on HBASE-17123: -- Was going over something else when found this:

[jira] [Commented] (HBASE-17123) Add postBulkLoadHFile variant that notifies the final paths for the hfiles

2016-11-21 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-17123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15684565#comment-15684565 ] Hudson commented on HBASE-17123: SUCCESS: Integrated in Jenkins build HBase-Trunk_matrix #1994 (See

[jira] [Commented] (HBASE-17123) Add postBulkLoadHFile variant that notifies the final paths for the hfiles

2016-11-20 Thread Anoop Sam John (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-17123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15682757#comment-15682757 ] Anoop Sam John commented on HBASE-17123: This is committed I think. There is one issue with the

[jira] [Commented] (HBASE-17123) Add postBulkLoadHFile variant that notifies the final paths for the hfiles

2016-11-20 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-17123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15682053#comment-15682053 ] Hudson commented on HBASE-17123: SUCCESS: Integrated in Jenkins build HBase-Trunk_matrix #1989 (See

[jira] [Commented] (HBASE-17123) Add postBulkLoadHFile variant that notifies the final paths for the hfiles

2016-11-18 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-17123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15678583#comment-15678583 ] Hadoop QA commented on HBASE-17123: --- | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote ||

[jira] [Commented] (HBASE-17123) Add postBulkLoadHFile variant that notifies the final paths for the hfiles

2016-11-18 Thread Ted Yu (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-17123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15678371#comment-15678371 ] Ted Yu commented on HBASE-17123: Test failure in TestSaslFanOutOneBlockAsyncDFSOutput was not related. >

[jira] [Commented] (HBASE-17123) Add postBulkLoadHFile variant that notifies the final paths for the hfiles

2016-11-18 Thread Ted Yu (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-17123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15678357#comment-15678357 ] Ted Yu commented on HBASE-17123: bq. deprecate the RegionObserver method in favor of the previous one

[jira] [Commented] (HBASE-17123) Add postBulkLoadHFile variant that notifies the final paths for the hfiles

2016-11-18 Thread Enis Soztutar (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-17123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15678349#comment-15678349 ] Enis Soztutar commented on HBASE-17123: --- The patch looks good. We want to also deprecate the

[jira] [Commented] (HBASE-17123) Add postBulkLoadHFile variant that notifies the final paths for the hfiles

2016-11-18 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-17123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15678282#comment-15678282 ] Hadoop QA commented on HBASE-17123: --- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem

[jira] [Commented] (HBASE-17123) Add postBulkLoadHFile variant that notifies the final paths for the hfiles

2016-11-18 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-17123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15677591#comment-15677591 ] Hadoop QA commented on HBASE-17123: --- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem

[jira] [Commented] (HBASE-17123) Add postBulkLoadHFile variant that notifies the final paths for the hfiles

2016-11-17 Thread Ted Yu (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-17123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15675527#comment-15675527 ] Ted Yu commented on HBASE-17123: SecureBulkLoadManager calls bulkLoadHFiles() of Region: {code}