[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16970676#comment-16970676
]
Hudson commented on HBASE-22480:
Results for branch branch-2
[build #2348 on
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16970639#comment-16970639
]
Hudson commented on HBASE-22480:
Results for branch branch-2.2
[build #687 on
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16970323#comment-16970323
]
Hudson commented on HBASE-22480:
Results for branch master
[build #1530 on
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16969817#comment-16969817
]
chenxu commented on HBASE-22480:
After reading the code, I have some doubt about
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16969730#comment-16969730
]
Zheng Hu commented on HBASE-22480:
--
The UT failure is unrelated to the patch, +1 to commit the patch.
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16969727#comment-16969727
]
Lijin Bin commented on HBASE-22480:
---
[~anoop.hbase] Thanks very much for the review.
> Get block
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16969252#comment-16969252
]
HBase QA commented on HBASE-22480:
--
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16969218#comment-16969218
]
HBase QA commented on HBASE-22480:
--
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16968988#comment-16968988
]
Lijin Bin commented on HBASE-22480:
---
Upload new patch to fix checkstyle.
> Get block from BlockCache
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16968982#comment-16968982
]
HBase QA commented on HBASE-22480:
--
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16968948#comment-16968948
]
HBase QA commented on HBASE-22480:
--
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16968843#comment-16968843
]
Lijin Bin commented on HBASE-22480:
---
attach again to trigger the HBaseQA
> Get block from BlockCache
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16968116#comment-16968116
]
Lijin Bin commented on HBASE-22480:
---
OK, make new patch for master and branch-2.2
> Get block from
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16968081#comment-16968081
]
Anoop Sam John commented on HBASE-22480:
bq.i find the latest patch do not applies, so i need to
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16968078#comment-16968078
]
Lijin Bin commented on HBASE-22480:
---
The patch apply to branch-2.2 but not master.
> Get block from
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16968067#comment-16968067
]
Lijin Bin commented on HBASE-22480:
---
[~anoop.hbase] in contrast,i find the latest patch do not
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16968058#comment-16968058
]
Anoop Sam John commented on HBASE-22480:
Not sure whether the latest patch still applies. But
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16850867#comment-16850867
]
HBase QA commented on HBASE-22480:
--
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16850516#comment-16850516
]
Anoop Sam John commented on HBASE-22480:
Patch LGTM. Great find.
> Get block from BlockCache
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16849620#comment-16849620
]
HBase QA commented on HBASE-22480:
--
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16849548#comment-16849548
]
HBase QA commented on HBASE-22480:
--
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16849375#comment-16849375
]
binlijin commented on HBASE-22480:
--
[~ram_krish]
{quote}
So here there is no negative ref counting na.
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16849371#comment-16849371
]
binlijin commented on HBASE-22480:
--
[~openinx], nice finding, upload a new patch which use you method
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16849319#comment-16849319
]
ramkrishna.s.vasudevan commented on HBASE-22480:
[~aoxiang]
So here there is no negative
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16849298#comment-16849298
]
Zheng Hu commented on HBASE-22480:
--
There're several place where we did not consider the release or
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16849284#comment-16849284
]
binlijin commented on HBASE-22480:
--
Yes, the seekToDataBlock() returns the same curBlock and after this
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16849089#comment-16849089
]
ramkrishna.s.vasudevan commented on HBASE-22480:
So here it means that the
{code}
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16848993#comment-16848993
]
HBase QA commented on HBASE-22480:
--
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16848922#comment-16848922
]
HBase QA commented on HBASE-22480:
--
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16848855#comment-16848855
]
binlijin commented on HBASE-22480:
--
I known the path loadBlockAndSeekToKey -> updateCurrentBlock ->
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16848814#comment-16848814
]
Zheng Hu commented on HBASE-22480:
--
Could we simplify the patch as the above I attached before ?
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16848773#comment-16848773
]
binlijin commented on HBASE-22480:
--
Yes, i also just thought of this we should not return this block
[
https://issues.apache.org/jira/browse/HBASE-22480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16848768#comment-16848768
]
Zheng Hu commented on HBASE-22480:
--
[~aoxiang], Nice finding, Thanks for your work. I'm also working
33 matches
Mail list logo