hbase-user mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ted Yu <yuzhih...@gmail.com>
Subject Re: Potential bugs in HTable In incrementColumnValue method
Date Tue, 09 Jun 2015 23:09:22 GMT
Did a quick search in HBase JIRA - no hit.

Jerry:
Mind logging one ?

Thanks

On Tue, Jun 9, 2015 at 3:30 PM, Andrew Purtell <apurtell@apache.org> wrote:

> Is there a JIRA for this?
>
> On Tue, Jun 9, 2015 at 11:15 AM, Ted Yu <yuzhihong@gmail.com> wrote:
>
> > Seems a bug to me w.r.t. interpretation of writeToWAL
> >
> > Cheers
> >
> > On Tue, Jun 9, 2015 at 10:50 AM, Jerry Lam <chilinglam@gmail.com> wrote:
> >
> > > Hi Vlad,
> > >
> > > I copied the code from HBase version 1.0.0.
> > > I first noticed it in version 0.98.6.
> > >
> > > We have codes that use HBase since 0.92. So some of the codes have not
> > been
> > > ported to the latest version therefore they are still using the
> > deprecated
> > > methods.
> > >
> > > The reason I'm asking is because I don't know if I should use SKIP_WAL
> to
> > > get the same semantic of writeToWAL (true). I'm doubting it because the
> > > name SKIP_WAL implies writeToWAL false. :)
> > >
> > > Best Regards,
> > >
> > > Jerry
> > >
> > >
> > >
> > > On Tue, Jun 9, 2015 at 12:03 PM, Ted Yu <yuzhihong@gmail.com> wrote:
> > >
> > > > I see code in this formation in 0.98 branch.
> > > >
> > > > Looking at the unit tests which exercise incrementColumnValue(), they
> > all
> > > > call:
> > > >   public long incrementColumnValue(final byte [] row, final byte []
> > > family,
> > > >       final byte [] qualifier, final long amount)
> > > > Possibly because the one mentioned by Jerry is deprecated.
> > > >
> > > > FYI
> > > >
> > > > On Tue, Jun 9, 2015 at 8:49 AM, Vladimir Rodionov <
> > > vladrodionov@gmail.com>
> > > > wrote:
> > > >
> > > > > Hi, Jerry
> > > > >
> > > > > Which version of HBase is it?
> > > > >
> > > > > -Vlad
> > > > >
> > > > > On Tue, Jun 9, 2015 at 8:05 AM, Jerry Lam <chilinglam@gmail.com>
> > > wrote:
> > > > >
> > > > > > Hi HBase community,
> > > > > >
> > > > > > Can anyone confirm that the method incrementColumnValue is
> > > implemented
> > > > > > correctly?
> > > > > >
> > > > > > I'm talking about mainly the deprecated method:
> > > > > >
> > > > > >  @Deprecated
> > > > > >
> > > > > >   @Override
> > > > > >
> > > > > >   public long incrementColumnValue(final byte [] row, final
byte
> []
> > > > > family,
> > > > > >
> > > > > >       final byte [] qualifier, final long amount, final boolean
> > > > > writeToWAL)
> > > > > >
> > > > > >   throws IOException {
> > > > > >
> > > > > >     return incrementColumnValue(row, family, qualifier, amount,
> > > > > >
> > > > > >       writeToWAL? Durability.SKIP_WAL: Durability.USE_DEFAULT);
> > > > > >
> > > > > >   }
> > > > > >
> > > > > >
> > > > > > Note from the above, if writeToWAL is true, Durability is set
to
> > > > > SKIP_WAL.
> > > > > >
> > > > > > It does not make sense to me so I'm asking if this might be
a
> > > potential
> > > > > > bug.
> > > > > >
> > > > > >
> > > > > > Best Regards,
> > > > > >
> > > > > >
> > > > > > Jerry
> > > > > >
> > > > >
> > > >
> > >
> >
>
>
>
> --
> Best regards,
>
>    - Andy
>
> Problems worthy of attack prove their worth by hitting back. - Piet Hein
> (via Tom White)
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message