Mailing-List: contact dev-help@accumulo.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@accumulo.apache.org
Received-SPF: softfail (nike.apache.org: transitioning domain of
 marc@accumulo.net does not designate 209.85.220.169 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CAOiJXP5cJ4-y-NZaAPLtUAG7_812P39eLmuXW8GRnUjUfE+NUg@mail.gmail.com>
References: 
 <CAOiJXP5cJ4-y-NZaAPLtUAG7_812P39eLmuXW8GRnUjUfE+NUg@mail.gmail.com>
Date: Tue, 13 Nov 2012 13:13:27 -0500
Message-ID: 
 <CALwRz8OpgL9_hW6eH4T=gWk2AK6Uha8m4vBbZx-FuG+To7YMVA@mail.gmail.com>
Subject: Re: Bug In Text Class - getLength() and getBytes().length are
 different.
From: Marc Parisi <marc@accumulo.net>
To: dev@accumulo.apache.org
Content-Type: multipart/alternative; boundary=20cf307f319ee0b56404ce645fc4

--20cf307f319ee0b56404ce645fc4
Content-Type: text/plain; charset=ISO-8859-1

That's expected. As per
http://docs.oracle.com/javase/6/docs/api/java/nio/Buffer.html, the byte
buffer is created by the encoder, where there is no guarantee that the
backing array's length would match the position ( or limit ).

On Tue, Nov 13, 2012 at 1:00 PM, David Medinets <david.medinets@gmail.com>wrote:

> The following code (the TextTest class) displays:
>
> cq: [5000000000000000]
> cq: [16]
> cq: [16]
> cq: [5000000000000000]
> cq: [16]
> cq: [17]
>
> You'll notice that the last two numbers are different, but they should
> both be 16. This bug affects Accumulo because of the following code in
> Mutation:
>
>   private void put(byte b[]) {
>     buffer.writeVLong(b.length);
>     buffer.add(b, 0, b.length);
>   }
>
>   private void put(Text t) {
>     buffer.writeVLong(t.getLength());
>     buffer.add(t.getBytes(), 0, t.getLength());
>   }
>
> I should be able to call either of the following to get the same
> result but I can't.
>
>   put("5000000000000000".getBytes());
>   put(new Text("5000000000000000"));
>
> Has anyone else run into this issue? Any workarounds or fixes?
>
> ----
>
> package com.codebits.accumulo;
>
> import org.apache.hadoop.io.Text;
>
> public class TextTest {
>
>   public static void main(String[] args) {
>     String s = "5000000000000000";
>     System.out.println("cq: [" + s + "]");
>     System.out.println("cq: [" + s.length() + "]");
>     System.out.println("cq: [" + s.getBytes().length + "]");
>
>     Text cq = new Text(s);
>     System.out.println("cq: [" + cq + "]");
>     System.out.println("cq: [" + cq.getLength() + "]");
>     System.out.println("cq: [" + cq.getBytes().length + "]");
>   }
>
> }
>

--20cf307f319ee0b56404ce645fc4--