bug in SequenceFile.sync()?

2013-06-24 Thread Christopher Ng
cross-posting this from cdh-users group where it received little interest: is there a bug in SequenceFile.sync()? This is from cdh4.3.0: /** Seek to the next sync mark past a given position.*/ public synchronized void sync(long position) throws IOException { if (position+SYNC_SIZE

Re: bug in SequenceFile.sync()?

2013-06-24 Thread Jean-Baptiste Onofré
Hi Christopher, indeed, I think that the noBufferedKeys and valuesDecompressed should be reset. Regards JB On 06/24/2013 11:20 AM, Christopher Ng wrote: cross-posting this from cdh-users group where it received little interest: is there a bug in SequenceFile.sync()? This is from cdh4.3.0

Re: bug in SequenceFile.sync()?

2013-06-24 Thread Colin McCabe
: is there a bug in SequenceFile.sync()? This is from cdh4.3.0: /** Seek to the next sync mark past a given position.*/ public synchronized void sync(long position) throws IOException { if (position+SYNC_SIZE = end) { seek(end); return; } if (position

Re: bug in SequenceFile.sync()?

2013-06-24 Thread Christopher Ng
for this. Colin Software Engineer, Cloudera On Mon, Jun 24, 2013 at 2:20 AM, Christopher Ng cng1...@gmail.com wrote: cross-posting this from cdh-users group where it received little interest: is there a bug in SequenceFile.sync()? This is from cdh4.3.0: /** Seek to the next sync mark