[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2017-03-09 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15903764#comment-15903764 ] stack commented on HBASE-15716: --- Thanks for reviving this one [~vincentpoon] > HRegion#RegionScannerImpl

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2017-03-09 Thread Vincent Poon (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15903597#comment-15903597 ] Vincent Poon commented on HBASE-15716: -- I did a quick test with the patch here. Using hbase pe

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-12-08 Thread Yu Li (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15731545#comment-15731545 ] Yu Li commented on HBASE-15716: --- Ping, this is something worth moving on according to our test result

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-12-05 Thread binlijin (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15721718#comment-15721718 ] binlijin commented on HBASE-15716: -- Looks like we can not use the patch for master version now? Mvcc

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-07-08 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15367838#comment-15367838 ] stack commented on HBASE-15716: --- This is part of a project that is trying to remove all or most points of

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-07-08 Thread Lars Hofhansl (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15367503#comment-15367503 ] Lars Hofhansl commented on HBASE-15716: --- What's the total (wall clock) runtime improvement? (just

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-07-07 Thread Hiroshi Ikeda (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15367154#comment-15367154 ] Hiroshi Ikeda commented on HBASE-15716: --- I realized I should have named the test class using the

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-07-05 Thread Hiroshi Ikeda (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15362226#comment-15362226 ] Hiroshi Ikeda commented on HBASE-15716: --- bq. I should make getReadPoint private. It is only for use

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-07-04 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15361997#comment-15361997 ] stack commented on HBASE-15716: --- Let me upload a new patch w/ your changes included. A few comments on the

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-07-04 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15361910#comment-15361910 ] stack commented on HBASE-15716: --- [~ikeda] Pardon me. That backport was so we had parity with hadoop rpc. It

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-07-04 Thread Hiroshi Ikeda (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15361878#comment-15361878 ] Hiroshi Ikeda commented on HBASE-15716: --- I found there is some changes on the master around this

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-07-04 Thread Hiroshi Ikeda (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15361009#comment-15361009 ] Hiroshi Ikeda commented on HBASE-15716: --- bq. You have tests that prove going backwards? It is

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-07-01 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15359682#comment-15359682 ] stack commented on HBASE-15716: --- Can you make a patch [~ikeda]? I tried looking at HBASE-14479 and

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-07-01 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15359023#comment-15359023 ] stack commented on HBASE-15716: --- Thanks for finding these items [~ikeda] You have tests that prove going

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-07-01 Thread Hiroshi Ikeda (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15358754#comment-15358754 ] Hiroshi Ikeda commented on HBASE-15716: --- That's bad. When there is no scanner, it is possible that

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-07-01 Thread Hiroshi Ikeda (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15358632#comment-15358632 ] Hiroshi Ikeda commented on HBASE-15716: --- Sorry, I found that getSmallestReadPoint() may go backward

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

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

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-06-30 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15358441#comment-15358441 ] stack commented on HBASE-15716: --- Issue that picks up some of the changes made in here but not committed as

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-06-30 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15358399#comment-15358399 ] stack commented on HBASE-15716: --- Ok. Changed my mind. The 005 patch has ONLY the [~ikeda] changes. I will

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-06-30 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15358380#comment-15358380 ] stack commented on HBASE-15716: --- I think it is fine. I went to hack on your class because I'd thought you'd

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-06-30 Thread Hiroshi Ikeda (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15358319#comment-15358319 ] Hiroshi Ikeda commented on HBASE-15716: --- {code} + } // Otherwise head.references is

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-06-30 Thread Hiroshi Ikeda (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15358316#comment-15358316 ] Hiroshi Ikeda commented on HBASE-15716: --- I meant, in Listener.doAccept, change {code} Reader reader

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

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

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-06-30 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15357027#comment-15357027 ] stack commented on HBASE-15716: --- On the files, looks very nice. Let me try it and report back. >

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-06-30 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15357020#comment-15357020 ] stack commented on HBASE-15716: --- This sounds interesting [~ikeda]. Let me see if I understand properly so I

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-06-29 Thread Hiroshi Ikeda (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15356363#comment-15356363 ] Hiroshi Ikeda commented on HBASE-15716: --- Now I find the menu item "Attach Files". But I feel it

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-06-29 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15355414#comment-15355414 ] stack commented on HBASE-15716: --- I assigned the issue to you [~ikeda] as per [~yangzhe1991] suggestion.

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-06-28 Thread Hiroshi Ikeda (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15354603#comment-15354603 ] Hiroshi Ikeda commented on HBASE-15716: --- I find I'm an administrator by the wheel icon, but still

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-06-28 Thread Phil Yang (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15354601#comment-15354601 ] Phil Yang commented on HBASE-15716: --- Try "assign to me" first, seems only assignee can attach files >

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-06-28 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15354592#comment-15354592 ] stack commented on HBASE-15716: --- [~ikeda] I made you an administrator. Try now. >

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-06-28 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15354589#comment-15354589 ] stack commented on HBASE-15716: --- You are logged in? > HRegion#RegionScannerImpl scannerReadPoints

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-06-28 Thread Hiroshi Ikeda (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15354577#comment-15354577 ] Hiroshi Ikeda commented on HBASE-15716: --- I still don't find a menu item or something to attach

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

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

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-06-28 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15353343#comment-15353343 ] stack commented on HBASE-15716: --- [~ikeda] Try now. I added you to contributors (access to JIRA was shutdown

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-06-28 Thread Hiroshi Ikeda (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15352818#comment-15352818 ] Hiroshi Ikeda commented on HBASE-15716: --- I tried some and created some tiny file but I just

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

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

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-06-28 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15352454#comment-15352454 ] stack commented on HBASE-15716: --- Address comment up on rb. Also add hashcode and equals to region and to

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-06-27 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15352396#comment-15352396 ] stack commented on HBASE-15716: --- Findbugs is because of recent commit on HBASE-14730. Pinged over there.

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

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

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

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

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

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

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-06-14 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15330047#comment-15330047 ] stack commented on HBASE-15716: --- One note is that there is also registration of Store Scanners with the

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-06-05 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15316153#comment-15316153 ] stack commented on HBASE-15716: --- Interesting observation was hacking out this lock, I ran then into my

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-05-01 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15266144#comment-15266144 ] stack commented on HBASE-15716: --- [~ikeda We continue conservative here, w/o change. I like your suggestion

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

2016-05-01 Thread Hiroshi Ikeda (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-15716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15266056#comment-15266056 ] Hiroshi Ikeda commented on HBASE-15716: --- {code} + // If the isolation level is

[jira] [Commented] (HBASE-15716) HRegion#RegionScannerImpl scannerReadPoints synchronization constrains random read

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