[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16592759#comment-16592759
]
Ted Yu commented on HBASE-15410:
It seems the following two fields of FilterListBase:
{code}
protected
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16592754#comment-16592754
]
Ted Yu commented on HBASE-15410:
Now that HBASE-19008 has been integrated, I wonder whether we can
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16221292#comment-16221292
]
Hudson commented on HBASE-15410:
SUCCESS: Integrated in Jenkins build HBase-2.0 #750 (See
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16221097#comment-16221097
]
Hudson commented on HBASE-15410:
FAILURE: Integrated in Jenkins build HBase-Trunk_matrix #3950 (See
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16220307#comment-16220307
]
Zheng Hu commented on HBASE-15410:
--
Pushed the addendum into master & branch-2. Thanks all.
> Utilize
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16220175#comment-16220175
]
Zheng Hu commented on HBASE-15410:
--
OK, If no other concern , I'll push it into master and branch-2.
>
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16220159#comment-16220159
]
Ted Yu commented on HBASE-15410:
Looks good
> Utilize the max seek value when all Filters in
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16219987#comment-16219987
]
Zheng Hu commented on HBASE-15410:
--
[~tedyu], Any concern about the addendum ?
> Utilize the max seek
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16219065#comment-16219065
]
Hudson commented on HBASE-15410:
FAILURE: Integrated in Jenkins build HBase-2.0 #748 (See
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16218758#comment-16218758
]
Hudson commented on HBASE-15410:
FAILURE: Integrated in Jenkins build HBase-Trunk_matrix #3948 (See
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16204426#comment-16204426
]
Hudson commented on HBASE-15410:
FAILURE: Integrated in Jenkins build HBase-2.0 #682 (See
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16204409#comment-16204409
]
Hudson commented on HBASE-15410:
FAILURE: Integrated in Jenkins build HBase-Trunk_matrix #3882 (See
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16204270#comment-16204270
]
Hadoop QA commented on HBASE-15410:
---
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16204109#comment-16204109
]
Chia-Ping Tsai commented on HBASE-15410:
v4 LGTM.
nit:
{code}
+import java.util.Collection;
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16204028#comment-16204028
]
Sean Busbey commented on HBASE-15410:
-
v4 looks fine.
> Utilize the max seek value when all Filters
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16204022#comment-16204022
]
Mike Drob commented on HBASE-15410:
---
You have some unused imports still hanging about. Otherwise LGTM.
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16204005#comment-16204005
]
Mike Drob commented on HBASE-15410:
---
Is it safe to use {{HashSet}} here? I took a look at a handful of
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16203578#comment-16203578
]
Ted Yu commented on HBASE-15410:
Ping [~busbey]
> Utilize the max seek value when all Filters in
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16202319#comment-16202319
]
Josh Elser commented on HBASE-15410:
bq. I don't expect an optimization as described in the release
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16201858#comment-16201858
]
Ted Yu commented on HBASE-15410:
[~busbey]:
What do you think of your own comment above ?
> Utilize the
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16200579#comment-16200579
]
Ted Yu commented on HBASE-15410:
>From
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16200343#comment-16200343
]
Sean Busbey commented on HBASE-15410:
-
still reviewing Ted.
> Utilize the max seek value when all
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16200316#comment-16200316
]
Ted Yu commented on HBASE-15410:
[~busbey]:
Do you have other comment ?
> Utilize the max seek value
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16198889#comment-16198889
]
Ted Yu commented on HBASE-15410:
The changes in TestFilterList are due to the optimization.
> Utilize
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16198887#comment-16198887
]
Sean Busbey commented on HBASE-15410:
-
Are the changes in TestFilterList intentional? I don't expect
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16198857#comment-16198857
]
Sean Busbey commented on HBASE-15410:
-
looking at this today. Do you have a proposed release note for
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16198713#comment-16198713
]
Ted Yu commented on HBASE-15410:
[~busbey]:
Do you want to take a look ?
> Utilize the max seek value
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16198249#comment-16198249
]
Hadoop QA commented on HBASE-15410:
---
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16198155#comment-16198155
]
Ted Yu commented on HBASE-15410:
Test added by HBASE-18957 is about MUST_PASS_ONE.
This optimization is
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16198107#comment-16198107
]
Hudson commented on HBASE-15410:
FAILURE: Integrated in Jenkins build HBase-Trunk_matrix #3859 (See
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16198090#comment-16198090
]
Hudson commented on HBASE-15410:
FAILURE: Integrated in Jenkins build HBase-2.0 #658 (See
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16197958#comment-16197958
]
Hudson commented on HBASE-15410:
FAILURE: Integrated in Jenkins build HBase-1.5 #89 (See
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16197494#comment-16197494
]
Sean Busbey commented on HBASE-15410:
-
FWIW, when HBASE-18957 lands there'll be a version of this
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16197334#comment-16197334
]
Ted Yu commented on HBASE-15410:
This JIRA appears in HBASE-18957
I will submit rebased patch to QA once
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16170909#comment-16170909
]
Hudson commented on HBASE-15410:
FAILURE: Integrated in Jenkins build HBase-1.5 #68 (See
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16157496#comment-16157496
]
Hudson commented on HBASE-15410:
FAILURE: Integrated in Jenkins build HBase-2.0 #473 (See
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16156945#comment-16156945
]
Hudson commented on HBASE-15410:
FAILURE: Integrated in Jenkins build HBase-Trunk_matrix #3674 (See
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16155740#comment-16155740
]
Chia-Ping Tsai commented on HBASE-15410:
Sorry for my carelessness. +1 again
> Utilize the max
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16155732#comment-16155732
]
Ted Yu commented on HBASE-15410:
In the patch already:
{code}
this.referenceCell = c;
+
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16155728#comment-16155728
]
Chia-Ping Tsai commented on HBASE-15410:
{code}
case SEEK_NEXT_USING_HINT:
-
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16155679#comment-16155679
]
Ted Yu commented on HBASE-15410:
I meant filterKeyValue() being called for different cells.
> Utilize
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16155675#comment-16155675
]
Chia-Ping Tsai commented on HBASE-15410:
bq. filterKeyValue() would be repeatedly called.
We
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16155658#comment-16155658
]
Chia-Ping Tsai commented on HBASE-15410:
Got it. +1
> Utilize the max seek value when all
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16155625#comment-16155625
]
Ted Yu commented on HBASE-15410:
filterKeyValue() would be repeatedly called.
A Set deduplicates the
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16155580#comment-16155580
]
Chia-Ping Tsai commented on HBASE-15410:
{code}
+ private Set seekHintFilter = new HashSet<>();
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16154708#comment-16154708
]
Hadoop QA commented on HBASE-15410:
---
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15435922#comment-15435922
]
Hadoop QA commented on HBASE-15410:
---
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15435220#comment-15435220
]
Ted Yu commented on HBASE-15410:
[~larsh]:
Mind taking a look ?
> Utilize the max seek value when all
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15332757#comment-15332757
]
Hadoop QA commented on HBASE-15410:
---
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15332391#comment-15332391
]
Jeff Zhang commented on HBASE-15410:
I did some testing against Ted's patch and was able to reproduce
[
https://issues.apache.org/jira/browse/HBASE-15410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15292598#comment-15292598
]
Ted Yu commented on HBASE-15410:
Need to investigate the following:
{code}
51 matches
Mail list logo