[ 
https://issues.apache.org/jira/browse/LUCENE-4258?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13509314#comment-13509314
 ] 

Michael McCandless commented on LUCENE-4258:
--------------------------------------------

Trying to catch up here ... I just have a bunch of random questions
(don't fully understand the patch yet):

Not sure why some files show as all deleted / all added lines, eg at
least FrozenBufferedDeletes.java.

Patch also has tabs, which should be spaces... (eg IndexWriter.java).

Why do we have FieldsUpdate.Operation.ADD_DOCUMENT?  It seems weird to
pass that to IW.updateFields?  Shouldn't apps just use
IW.addDocument?

Why do we need SegmentInfoReader.readFilesList?  It seems like it's
only privately used inside the codec?  I'm confused why the "normal"
file tracking we have on write is insufficient... oh I see, a single
SegmentInfo references all stacked segments too?  But since they are
written "later" their files won't be automatically tracked ... ok.  I
wonder if each stacked segment should get its own SegmentInfo, linked
to the base segment...

It looks like merge policies don't yet know about / target stacked
segments ...

It seems like we don't invert the document updates until the updates
are applied?  Ie, we just buffer the IndexableField provided by the
user and when it's time to apply updates, we then analyzing/invert?
How do we track RAM in this case?  (Eg the field could be something
arbitrary, eg pre-tokenized).  Another option is to do the invert and
buffer the resulting postings, and then later "replay" them (remapping
docIDs) when it's time to apply.

Why does StoredFieldsReader.visitDocument need a Set for ignored
fields?

                
> Incremental Field Updates through Stacked Segments
> --------------------------------------------------
>
>                 Key: LUCENE-4258
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4258
>             Project: Lucene - Core
>          Issue Type: Improvement
>          Components: core/index
>            Reporter: Sivan Yogev
>         Attachments: IncrementalFieldUpdates.odp, 
> LUCENE-4258-API-changes.patch, LUCENE-4258.r1410593.patch, 
> LUCENE-4258.r1412262.patch, LUCENE-4258.r1416438.patch, 
> LUCENE-4258.r1416617.patch
>
>   Original Estimate: 2,520h
>  Remaining Estimate: 2,520h
>
> Shai and I would like to start working on the proposal to Incremental Field 
> Updates outlined here (http://markmail.org/message/zhrdxxpfk6qvdaex).

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

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

Reply via email to