Mailing-List: contact dev-help@hbase.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@hbase.apache.org
MIME-Version: 1.0
Sender: saint.ack@gmail.com
In-Reply-To: <CAA4mzo=QX=xW6_2+_HqjhF5Z1BMJzRNQV2b8rxh26SpaG5GU5g@mail.gmail.com>
References: <pony-51ef25fd0a3611d18ffb478ae2ae610bbbd2432c-1cdb7bba9143eda2ba9f5b50aba58708ef7a6507@dev.hbase.apache.org>
 <CADcMMgGsHWMSXM64iBpSjvx42AJP061yfeAY1Y7-4bRf4yycww@mail.gmail.com>
 <pony-51ef25fd0a3611d18ffb478ae2ae610bbbd2432c-bf9b9373bbace91b223e29d1652fbee5746a51e9@dev.hbase.apache.org>
 <CAA4mzok-6FdnpSChSsHteUj2-6_U_6eCUOm5HM6gNGZRiU+Zow@mail.gmail.com>
 <pony-51ef25fd0a3611d18ffb478ae2ae610bbbd2432c-e9b9ec61485e151132bd6c9b981ad5ea9724fac1@dev.hbase.apache.org>
 <CAA4mzo=QX=xW6_2+_HqjhF5Z1BMJzRNQV2b8rxh26SpaG5GU5g@mail.gmail.com>
From: Stack <stack@duboce.net>
Date: Wed, 20 Sep 2017 09:41:19 -0700
Message-ID: <CADcMMgHYW5m=EKxN+KPzQ2H68qi2Cm2sPRd-5YoXyTUTQzNEzw@mail.gmail.com>
Subject: Re: [DISCUSS] Introduce the limited private to filter
To: HBase Dev List <dev@hbase.apache.org>
Content-Type: multipart/alternative; boundary="f403043a173c415d840559a1a560"
archived-at: Wed, 20 Sep 2017 16:41:32 -0000

--f403043a173c415d840559a1a560
Content-Type: text/plain; charset="UTF-8"

Thanks Dave for chiming in.

I like the suggestion Chia-Ping of LimittedPrivate + Stable.

St.Ack

On Wed, Sep 20, 2017 at 6:51 AM, Dave Latham <latham@davelink.net> wrote:

> Thanks, Chia-Ping.  I don't want to hold up improvements in HBase internals
> on behalf of Filters.  Changing at major releases sounds reasonable.
>
> On Wed, Sep 20, 2017 at 12:32 AM, Chia-Ping Tsai <chia7712@apache.org>
> wrote:
>
> > An alternative is that we make Filter IA.LimitedPrivate and
> > InterfaceStability.Stable, so we guarantee the FIlter's APIs won't be
> > changed in the patch/minor release. It seems to me the changing Filter's
> IA
> > declaration is necessary. The filter's APIs deeply embedded in our read
> > path and thus the filter may hinder our improvement in given major
> release
> > because we have to do the proper deprecation cycle for IA.Public. I DON'T
> > want to change the Filter's APIs at the present time, but we should get
> > everything ready beforehand.
> >
> > On 2017-09-19 00:41, Dave Latham <latham@davelink.net> wrote:
> > > Speaking on behalf of one HBase deployment, we do rely on custom
> filters,
> > > though have so far stayed away from more internal customizations such
> as
> > > co-processors.  We've gotten the sense over the years that Filters were
> > > fairly stable and seemed more reliable in that sense.  I'd be sad if a
> > > change like this meant that more caution will need to be used in order
> to
> > > rely on Filters.  I understand that some cleanup may need to happen
> (e.g.
> > > HBASE-13346) but hope that we can still be conservative in breaking the
> > > Filter apis.
> > >
> > > On Sat, Sep 16, 2017 at 7:27 PM, Chia-Ping Tsai <chia7712@apache.org>
> > wrote:
> > >
> > > > hi stack
> > > > I have filed https://issues.apache.org/jira/browse/HBASE-18811. FYI.
> > > >
> > > > On 2017-09-17 05:31, Stack <stack@duboce.net> wrote:
> > > > > It is an oversight that Filters are not annotated as
> > (limited)private. We
> > > > > are unable to guarantee them what public entails given their design
> > is as
> > > > > yet imperfect and that they are interpolated at points subject to
> > change.
> > > > >
> > > > > +1 on taking them limited private in 2.0.0.
> > > > >
> > > > > Thanks for bringing this up Chia-Ping Tsai. Apt.
> > > > >
> > > > > St.Ack
> > > > >
> > > > >
> > > > > On Sat, Sep 16, 2017 at 4:02 AM, Chia-Ping Tsai <
> chia7712@apache.org
> > >
> > > > wrote:
> > > > >
> > > > > > Hi, Folks!
> > > > > >
> > > > > > We have many powerful callback functions to help user to build
> > amazing
> > > > > > application/services. The most of functions are declared as
> > > > > > IA.LimitedPrivate excluding the filters. As i see it, the
> > > > IA.LimitedPrivate
> > > > > > will make the improvement of filter more flexible. Also, we can
> > > > introduce
> > > > > > more server-side components to filters.
> > > > > >
> > > > > > https://issues.apache.org/jira/browse/HBASE-9529 had already
> left
> > the
> > > > > > TODO "add filter limited private level" on FilterBase. I feel it
> is
> > > > time to
> > > > > > discuss it again.
> > > > > >
> > > > > > Thanks,
> > > > > > Chia-Ping Tsai
> > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
>

--f403043a173c415d840559a1a560--