You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@lucene.apache.org by Jason Rutherglen <ja...@gmail.com> on 2011/01/12 19:59:23 UTC

SegmentInfo clone

Is it intentional that SegmentInfo.segmentCodecs isn't cloned?  When
SI is cloned, then sizeInBytes fails with an NPE.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org


Re: SegmentInfo clone

Posted by Jason Rutherglen <ja...@gmail.com>.
> it is set on DocumentsWriter#flush though

Thanks!  I just skip segmentCodecs if it's null, for now.

On Wed, Jan 12, 2011 at 11:05 AM, Simon Willnauer
<si...@googlemail.com> wrote:
> On Wed, Jan 12, 2011 at 8:03 PM, Jason Rutherglen
> <ja...@gmail.com> wrote:
>> Sorry, that's incorrect, SegmentInfo.files is NPE'ing on segmentCodecs
>> because it's never set (in trunk).
> it is set on DocumentsWriter#flush though
>
> simon
>>
>> On Wed, Jan 12, 2011 at 10:59 AM, Jason Rutherglen
>> <ja...@gmail.com> wrote:
>>> Is it intentional that SegmentInfo.segmentCodecs isn't cloned?  When
>>> SI is cloned, then sizeInBytes fails with an NPE.
>>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
>> For additional commands, e-mail: dev-help@lucene.apache.org
>>
>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
> For additional commands, e-mail: dev-help@lucene.apache.org
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org


Re: SegmentInfo clone

Posted by Simon Willnauer <si...@googlemail.com>.
On Wed, Jan 12, 2011 at 8:03 PM, Jason Rutherglen
<ja...@gmail.com> wrote:
> Sorry, that's incorrect, SegmentInfo.files is NPE'ing on segmentCodecs
> because it's never set (in trunk).
it is set on DocumentsWriter#flush though

simon
>
> On Wed, Jan 12, 2011 at 10:59 AM, Jason Rutherglen
> <ja...@gmail.com> wrote:
>> Is it intentional that SegmentInfo.segmentCodecs isn't cloned?  When
>> SI is cloned, then sizeInBytes fails with an NPE.
>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
> For additional commands, e-mail: dev-help@lucene.apache.org
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org


Re: SegmentInfo clone

Posted by Jason Rutherglen <ja...@gmail.com>.
Sorry, that's incorrect, SegmentInfo.files is NPE'ing on segmentCodecs
because it's never set (in trunk).

On Wed, Jan 12, 2011 at 10:59 AM, Jason Rutherglen
<ja...@gmail.com> wrote:
> Is it intentional that SegmentInfo.segmentCodecs isn't cloned?  When
> SI is cloned, then sizeInBytes fails with an NPE.
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org