[jira] [Commented] (PHOENIX-4053) Lock row exclusively when necessary for mutable secondary indexing

2017-08-15 Thread James Taylor (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16127968#comment-16127968 ] James Taylor commented on PHOENIX-4053: --- I think we should just proceed with a 4.12 that'll

[jira] [Commented] (PHOENIX-4053) Lock row exclusively when necessary for mutable secondary indexing

2017-08-15 Thread Samarth Jain (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16127906#comment-16127906 ] Samarth Jain commented on PHOENIX-4053: --- [~jamestaylor] - I don't see the patch available in our

[jira] [Commented] (PHOENIX-4053) Lock row exclusively when necessary for mutable secondary indexing

2017-07-31 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16107848#comment-16107848 ] Hudson commented on PHOENIX-4053: - FAILURE: Integrated in Jenkins build Phoenix-master #1723 (See

[jira] [Commented] (PHOENIX-4053) Lock row exclusively when necessary for mutable secondary indexing

2017-07-31 Thread Samarth Jain (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16107676#comment-16107676 ] Samarth Jain commented on PHOENIX-4053: --- Thanks for the explanation, James. +1 to the patch after

[jira] [Commented] (PHOENIX-4053) Lock row exclusively when necessary for mutable secondary indexing

2017-07-31 Thread Andrew Purtell (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16107656#comment-16107656 ] Andrew Purtell commented on PHOENIX-4053: - Looked at the v6 patch. I like that you're passing

[jira] [Commented] (PHOENIX-4053) Lock row exclusively when necessary for mutable secondary indexing

2017-07-31 Thread James Taylor (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16107614#comment-16107614 ] James Taylor commented on PHOENIX-4053: --- The lockManager.lockRow() call throws an unchecked

[jira] [Commented] (PHOENIX-4053) Lock row exclusively when necessary for mutable secondary indexing

2017-07-31 Thread Samarth Jain (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16107602#comment-16107602 ] Samarth Jain commented on PHOENIX-4053: --- [~jamestaylor], Regarding your comment here: {code} +

[jira] [Commented] (PHOENIX-4053) Lock row exclusively when necessary for mutable secondary indexing

2017-07-31 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16107564#comment-16107564 ] Hadoop QA commented on PHOENIX-4053: {color:red}-1 overall{color}. Here are the results of testing

[jira] [Commented] (PHOENIX-4053) Lock row exclusively when necessary for mutable secondary indexing

2017-07-30 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16106744#comment-16106744 ] Hadoop QA commented on PHOENIX-4053: {color:red}-1 overall{color}. Here are the results of testing

[jira] [Commented] (PHOENIX-4053) Lock row exclusively when necessary for mutable secondary indexing

2017-07-30 Thread James Taylor (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16106705#comment-16106705 ] James Taylor commented on PHOENIX-4053: --- Looks like HBASE-17210 isn't available in 1.3. I could

[jira] [Commented] (PHOENIX-4053) Lock row exclusively when necessary for mutable secondary indexing

2017-07-30 Thread Andrew Purtell (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16106696#comment-16106696 ] Andrew Purtell commented on PHOENIX-4053: - ConcurrentMutationsIT needs an ASF header. A concern

[jira] [Commented] (PHOENIX-4053) Lock row exclusively when necessary for mutable secondary indexing

2017-07-30 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16106695#comment-16106695 ] Hadoop QA commented on PHOENIX-4053: {color:red}-1 overall{color}. Here are the results of testing

[jira] [Commented] (PHOENIX-4053) Lock row exclusively when necessary for mutable secondary indexing

2017-07-30 Thread Andrew Purtell (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16106684#comment-16106684 ] Andrew Purtell commented on PHOENIX-4053: - I linked the three relevant HBase JIRAs I know of

[jira] [Commented] (PHOENIX-4053) Lock row exclusively when necessary for mutable secondary indexing

2017-07-30 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16106671#comment-16106671 ] Hadoop QA commented on PHOENIX-4053: {color:red}-1 overall{color}. Here are the results of testing

[jira] [Commented] (PHOENIX-4053) Lock row exclusively when necessary for mutable secondary indexing

2017-07-29 Thread Samarth Jain (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16106322#comment-16106322 ] Samarth Jain commented on PHOENIX-4053: --- [~jamestaylor], patch looks good for the most part. Below

[jira] [Commented] (PHOENIX-4053) Lock row exclusively when necessary for mutable secondary indexing

2017-07-29 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16106286#comment-16106286 ] Hadoop QA commented on PHOENIX-4053: {color:red}-1 overall{color}. Here are the results of testing

[jira] [Commented] (PHOENIX-4053) Lock row exclusively when necessary for mutable secondary indexing

2017-07-29 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16106271#comment-16106271 ] Hadoop QA commented on PHOENIX-4053: {color:red}-1 overall{color}. Here are the results of testing

[jira] [Commented] (PHOENIX-4053) Lock row exclusively when necessary for mutable secondary indexing

2017-07-28 Thread James Taylor (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16105837#comment-16105837 ] James Taylor commented on PHOENIX-4053: --- Need new patch as the row locks need to be in place from

[jira] [Commented] (PHOENIX-4053) Lock row exclusively when necessary for mutable secondary indexing

2017-07-28 Thread James Taylor (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-4053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16105649#comment-16105649 ] James Taylor commented on PHOENIX-4053: --- Confirmed that row lock became a read lock from HBase