[jira] [Commented] (PHOENIX-3789) Execute cross region index maintenance calls outside of row lock

[ https://issues.apache.org/jira/browse/PHOENIX-3789?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15969615#comment-15969615 ] Hadoop QA commented on PHOENIX-3789: {color:red}-1 overall{color}. Here are the results of testing

[jira] [Commented] (PHOENIX-3789) Execute cross region index maintenance calls outside of row lock

[ https://issues.apache.org/jira/browse/PHOENIX-3789?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15969610#comment-15969610 ] Andrew Purtell commented on PHOENIX-3789: - Ok. So there's only the pre hook thing to look at,

[jira] [Commented] (PHOENIX-3789) Execute cross region index maintenance calls outside of row lock

[ https://issues.apache.org/jira/browse/PHOENIX-3789?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15969593#comment-15969593 ] James Taylor commented on PHOENIX-3789: --- bq. If the mutation's durability is SKIP_WAL then Indexer

[jira] [Commented] (PHOENIX-3789) Execute cross region index maintenance calls outside of row lock

[ https://issues.apache.org/jira/browse/PHOENIX-3789?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15969573#comment-15969573 ] Andrew Purtell commented on PHOENIX-3789: - [~giacomotaylor] [~gjacoby] Wouldn't it be fair to

[jira] [Commented] (PHOENIX-3789) Execute cross region index maintenance calls outside of row lock

[ https://issues.apache.org/jira/browse/PHOENIX-3789?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15969568#comment-15969568 ] Geoffrey Jacoby commented on PHOENIX-3789: -- Thanks, [~apurtell] and [~jamestaylor]. +1 from me

[jira] [Commented] (PHOENIX-3789) Execute cross region index maintenance calls outside of row lock

[ https://issues.apache.org/jira/browse/PHOENIX-3789?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15969563#comment-15969563 ] James Taylor commented on PHOENIX-3789: --- The preMutateBatch only makes cross region calls if WAL

[jira] [Commented] (PHOENIX-3789) Execute cross region index maintenance calls outside of row lock

[ https://issues.apache.org/jira/browse/PHOENIX-3789?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15969561#comment-15969561 ] Andrew Purtell commented on PHOENIX-3789: - If the mutation's durability is SKIP_WAL then Indexer

[jira] [Commented] (PHOENIX-3789) Execute cross region index maintenance calls outside of row lock

[ https://issues.apache.org/jira/browse/PHOENIX-3789?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15969549#comment-15969549 ] Geoffrey Jacoby commented on PHOENIX-3789: -- It appears that the preMutateBatch code can make

[jira] [Commented] (PHOENIX-3789) Execute cross region index maintenance calls outside of row lock

[ https://issues.apache.org/jira/browse/PHOENIX-3789?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15969540#comment-15969540 ] Andrew Purtell commented on PHOENIX-3789: - bq. I see in Indexer there are also overrides in

[jira] [Commented] (PHOENIX-3789) Execute cross region index maintenance calls outside of row lock

[ https://issues.apache.org/jira/browse/PHOENIX-3789?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15969537#comment-15969537 ] Geoffrey Jacoby commented on PHOENIX-3789: -- I see in Indexer there are also overrides in

[jira] [Commented] (PHOENIX-3789) Execute cross region index maintenance calls outside of row lock

[ https://issues.apache.org/jira/browse/PHOENIX-3789?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15969506#comment-15969506 ] Andrew Purtell commented on PHOENIX-3789: - FWIW, after HBASE-17210 at least rowlock acquisition

[jira] [Commented] (PHOENIX-3789) Execute cross region index maintenance calls outside of row lock

[ https://issues.apache.org/jira/browse/PHOENIX-3789?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15969503#comment-15969503 ] Andrew Purtell commented on PHOENIX-3789: - +1 [~giacomotaylor] I think the change is needed so