Mailing-List: contact dev-help@hbase.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@hbase.apache.org
MIME-Version: 1.0
In-Reply-To: <3d51a52e-835c-b2c2-1767-7328120ae268@apache.org>
References: <CACc8XkELFoB+6w9CxpH_7QX-_wE4ChWZGr9DehYgDFW3q7wEjw@mail.gmail.com>
 <CAAAYAnOZTzH0K5G29=kxEu180CMQke1ZypmswuPxOeAQNkKJFw@mail.gmail.com>
 <CALte62ykKhBPCk+B7_5R2ubrQaMSeWPAp_=CfA3dya=Sqbuqhw@mail.gmail.com>
 <CAB8hLakLdMfRx+aMxcuZbgL9KFehABvb2BVPnjAMcqr95CEmsg@mail.gmail.com>
 <CACc8XkE0xreAcQwvV=L-hPrF9BbxVe_L4=amHJSCkBRyvsaO+A@mail.gmail.com>
 <CAAAYAnNQdXGr5-kP-J=d0eizF9wRLGrYkeqjd4Gk89ce8xgjsg@mail.gmail.com>
 <CAOtJ30o3N9wAHmsaj04nPoCQhDhoFf9wkeNqwgrFbm2JcRUMqg@mail.gmail.com>
 <CACc8XkGoHB=YAdifJ9vuB2SRKJdeOmC5zjrWfykwdMwMwmvBWw@mail.gmail.com>
 <CADcMMgG_RChg+fFv++s+s8TvkXPS37ESQr2LE3L3VACB9K0+TA@mail.gmail.com>
 <CADcMMgFVXBb2TzYJpRG-qyy4tGgDSSnJz=XPSUqf7FoydF7+Rg@mail.gmail.com> <3d51a52e-835c-b2c2-1767-7328120ae268@apache.org>
From: =?UTF-8?B?5byg6ZOOKER1byBaaGFuZyk=?= <palomino219@gmail.com>
Date: Wed, 25 Oct 2017 20:32:11 +0800
Message-ID: <CAAAYAnPpH+_4CcwO5nt30EeVfrT1hcJkj7xWpjNjgnc177k9=w@mail.gmail.com>
Subject: Re: [DISCUSS] Merge FilterList Improvement - Branch HBASE-18410
To: dev@hbase.apache.org
Content-Type: multipart/alternative; boundary="001a11439ec2ad4b0e055c5e3e18"
archived-at: Wed, 25 Oct 2017 12:32:17 -0000

--001a11439ec2ad4b0e055c5e3e18
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

+1. Just done another round of rebase and force push, it is clean. Let me
do the merge now.

Thanks all for taking care of this.

2017-10-25 11:26 GMT+08:00 Josh Elser <elserj@apache.org>:

> +1 IMO, go for it -- enough eyes have been had on the last little bit.
>
>
> On 10/24/17 11:05 PM, stack wrote:
>
>> The boys are almost ready to merge. They were going to run a VOTE. I tol=
d
>> them not to bother. There is enough support behind the merge they don't
>> need a vote IMO. If you think otherwise, speak up.
>>
>> Thanks,
>>
>> St.Ack
>> P.S. Release note on HBASE-18410 will have summary of changes that come =
in
>> on the merge. Thanks.
>>
>> On Sat, Oct 21, 2017 at 6:37 PM, stack <saint.ack@gmail.com> wrote:
>>
>> +1 on merge then sorting out the mess. I want to include this stuff in
>>> alpha 4.
>>>
>>> Thanks,
>>> S
>>>
>>> On Oct 20, 2017 01:18, "OpenInx" <openinx@gmail.com> wrote:
>>>
>>> Fine,  I opened jira HBASE-19057 for it.
>>>>
>>>> On Fri, Oct 20, 2017 at 4:13 PM, Anoop John <anoop.hbase@gmail.com>
>>>> wrote:
>>>>
>>>> +1 for opening an issue and handle the merge as part of that.
>>>>> I did a review of the new classes for the FilterList impl..  Great
>>>>> cleanup work.  Very easy to read and understand the code now..
>>>>> Have few comments specially on FilterListWithOR.   If u can raise a
>>>>> merge issue, can comment down that.  If needed can open up subtasks
>>>>> can then handle.
>>>>>
>>>>> -Anoop-
>>>>>
>>>>> On Fri, Oct 20, 2017 at 12:01 PM, =E5=BC=A0=E9=93=8E(Duo Zhang) <palo=
mino219@gmail.com
>>>>> >
>>>>> wrote:
>>>>>
>>>>>> Open a issue to track the merging work? Let's do a rebase and more
>>>>>>
>>>>> testing
>>>>>
>>>>>> in that issue. And then come back and open a vote.
>>>>>>
>>>>>> 2017-10-20 11:48 GMT+08:00 OpenInx <openinx@gmail.com>:
>>>>>>
>>>>>>   I believe there are several UTs which we want them to be the
>>>>>>>>
>>>>>>> guard of
>>>>
>>>>> merging?
>>>>>>> What is the situation of these UTs?
>>>>>>>
>>>>>>> The UT is TestFilterListOnMini which make sure the filter list with
>>>>>>>
>>>>>> family
>>>>>
>>>>>> filter in it works fine,  it's enable in branch HBASE-18410 now .
>>>>>>>
>>>>>> see
>>>>
>>>>> https://issues.apache.org/jira/browse/HBASE-18977.
>>>>>>>
>>>>>>> Some of recent patches, such as HBASE-18368-HBASE-18410.v2.patch,
>>>>>>>>
>>>>>>> didn't get
>>>>>>> proper QA run
>>>>>>>
>>>>>>> All of the UT passed except one failed case , and this failed case =
is
>>>>>>> unrelated.   see
>>>>>>> https://builds.apache.org/job/PreCommit-HBASE-Build/9157/testReport=
/
>>>>>>>
>>>>>>> This is a incompatible change for filter developer ?
>>>>>>>>
>>>>>>>
>>>>>>> Yes, I think so.
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> On Fri, Oct 20, 2017 at 9:37 AM, Guanghao Zhang <zghaobac@gmail.com=
>
>>>>>>> wrote:
>>>>>>>
>>>>>>> HBASE-18368-HBASE-18410.v2.patch got a proper QA
>>>>>>>> run: PreCommit-HBASE-Build/9157. And the failed ut is not related.
>>>>>>>>
>>>>>>> So
>>>>
>>>>> I
>>>>>
>>>>>> committed it to branch HBASE-18410...
>>>>>>>> HBASE-18368 changed the javadoc of NEXT_ROW. This is a incompatibl=
e
>>>>>>>>
>>>>>>> change
>>>>>>>
>>>>>>>> for filter developer?
>>>>>>>>
>>>>>>>> 2017-10-20 9:14 GMT+08:00 Ted Yu <yuzhihong@gmail.com>:
>>>>>>>>
>>>>>>>> Some of recent patches, such as HBASE-18368-HBASE-18410.v2.pat
>>>>>>>>>
>>>>>>>> ch,
>>>>
>>>>> didn't
>>>>>>>
>>>>>>>> get proper QA run (due to precommit disruption).
>>>>>>>>>
>>>>>>>>> We'd better get several good QA runs.
>>>>>>>>>
>>>>>>>>> Cheers
>>>>>>>>>
>>>>>>>>> On Thu, Oct 19, 2017 at 5:38 PM, =E5=BC=A0=E9=93=8E(Duo Zhang) <
>>>>>>>>>
>>>>>>>> palomino219@gmail.com>
>>>>>
>>>>>> wrote:
>>>>>>>>>
>>>>>>>>> I believe there are several UTs which we want them to be the
>>>>>>>>>>
>>>>>>>>> guard of
>>>>>
>>>>>> merging? What is the situation of these UTs?
>>>>>>>>>>
>>>>>>>>>> Thanks.
>>>>>>>>>>
>>>>>>>>>> 2017-10-20 8:33 GMT+08:00 OpenInx <openinx@gmail.com>:
>>>>>>>>>>
>>>>>>>>>> Hi All :
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> All subtasks have been resolved except HBASE-18993
>>>>>>>>>>> <https://issues.apache.org/jira/browse/HBASE-18993>, I think
>>>>>>>>>>>
>>>>>>>>>> it's
>>>>>
>>>>>> time
>>>>>>>>
>>>>>>>>> to
>>>>>>>>>>
>>>>>>>>>>> merge HBASE-18410 <https://issues.apache.org/
>>>>>>>>>>>
>>>>>>>>>> jira/browse/HBASE-18410
>>>>>>>
>>>>>>>>
>>>>>>>>> branch
>>>>>>>>>>> to master now.
>>>>>>>>>>>
>>>>>>>>>>> Any concerns ?
>>>>>>>>>>>
>>>>>>>>>>> Thanks.
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> --
>>>>>>> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D
>>>>>>> Openinx  blog : http://openinx.github.io
>>>>>>>
>>>>>>> TO BE A GREAT HACKER !
>>>>>>> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D
>>>>>>>
>>>>>>>
>>>>>
>>>>
>>>>
>>>> --
>>>> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D
>>>> Openinx  blog : http://openinx.github.io
>>>>
>>>> TO BE A GREAT HACKER !
>>>> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D
>>>>
>>>>
>>>
>>

--001a11439ec2ad4b0e055c5e3e18--