[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization costs

2016-04-28 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15262438#comment-15262438 ] stack commented on HBASE-15716: --- Ok. Putting this aside for now. We could apply a patch tha

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization costs

2016-04-28 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15262403#comment-15262403 ] stack commented on HBASE-15716: --- bq. Calculating the smallest readpoint can be called by an

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization costs

2016-04-27 Thread Hiroshi Ikeda (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15261638#comment-15261638 ] Hiroshi Ikeda commented on HBASE-15716: --- {quote} Can you say more please? I do not

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization costs

2016-04-27 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15261587#comment-15261587 ] Hadoop QA commented on HBASE-15716: --- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vo

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization costs

2016-04-27 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15261585#comment-15261585 ] stack commented on HBASE-15716: --- Thank you the helpful review [~ikeda] Do you a fault other

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization costs

2016-04-27 Thread Hiroshi Ikeda (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15261563#comment-15261563 ] Hiroshi Ikeda commented on HBASE-15716: --- Quite strictly speaking, there is a very v

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization costs

2016-04-27 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15261248#comment-15261248 ] Hadoop QA commented on HBASE-15716: --- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vo

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization costs

2016-04-27 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15261068#comment-15261068 ] Hadoop QA commented on HBASE-15716: --- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vo

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization costs

2016-04-27 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15260556#comment-15260556 ] stack commented on HBASE-15716: --- It think the patches before v3 have a hole. v3 looks bette

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization costs

2016-04-27 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15260550#comment-15260550 ] stack commented on HBASE-15716: --- Sorry, the order in which my comments went in may confuse.

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization costs

2016-04-27 Thread Lars Hofhansl (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15260531#comment-15260531 ] Lars Hofhansl commented on HBASE-15716: --- +1 on the earlier patch that just removed

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization costs

2016-04-27 Thread Lars Hofhansl (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15260518#comment-15260518 ] Lars Hofhansl commented on HBASE-15716: --- Yeah, I think that should work. Was thin

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization costs

2016-04-27 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15260340#comment-15260340 ] stack commented on HBASE-15716: --- [~mantonov] It should go into 1.3 sir if I can get [~lhofh

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization costs

2016-04-27 Thread Mikhail Antonov (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15259763#comment-15259763 ] Mikhail Antonov commented on HBASE-15716: - [~stack] if you're testing on branch-1

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization costs

2016-04-26 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15259385#comment-15259385 ] stack commented on HBASE-15716: --- bq. The problem is that adding a readpoint and finding the

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization costs

2016-04-26 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15259371#comment-15259371 ] stack commented on HBASE-15716: --- Smile. Relies on the fact that as we create new scanners,

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization costs

2016-04-26 Thread Lars Hofhansl (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15259354#comment-15259354 ] Lars Hofhansl commented on HBASE-15716: --- How do you know it works? :) > HRegion#Re

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization costs

2016-04-26 Thread Lars Hofhansl (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15259350#comment-15259350 ] Lars Hofhansl commented on HBASE-15716: --- Hmm... As is we need the synchronized I th

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization costs

2016-04-26 Thread Lars Hofhansl (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15259271#comment-15259271 ] Lars Hofhansl commented on HBASE-15716: --- Looking now. > HRegion#RegionScannerImpl