[
https://issues.apache.org/jira/browse/LUCENE-2856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12980547#action_12980547
]
Shai Erera commented on LUCENE-2856:
------------------------------------
I think start/endTime can be long and not Long?
Maybe instead of adding init/start/endTime to OneMerge, you can pass a 'time'
parameter to MergeEvent. So its signature will be MergeEvent(Type type, long
time, OneMerge merge). The 'time' parameter can then be interpreted according
to Type.
Even better, I think you can remove that parameter entirely, and have
MergeEvent call System.currentTimeMillis() and set its internal member
according to Type?
About this:
{code}
public long getMergeSegmentSize(boolean includeDocStores) throws IOException {
// nocommit: cache this?
return mergeInfo.sizeInBytes(includeDocStores);
}
{code}
SegmentInfo caches sizeInBytes, so I think the 'nocommit' can go away.
However, I did notice that SegmentInfo's cache is potentially buggy -- it
doesn't take into account 'includeDocStores'. I.e., if you call it once w/
'false' and then with 'true' (or vice versa), you won't get the right
sizeInBytes. I'll open a separate issue to fix this.
> Create IndexWriter event listener, specifically for merges
> ----------------------------------------------------------
>
> Key: LUCENE-2856
> URL: https://issues.apache.org/jira/browse/LUCENE-2856
> Project: Lucene - Java
> Issue Type: Improvement
> Components: Index
> Affects Versions: 4.0
> Reporter: Jason Rutherglen
> Attachments: LUCENE-2856.patch
>
>
> The issue will allow users to monitor merges occurring within IndexWriter
> using a callback notifier event listener. This can be used by external
> applications such as Solr to monitor large segment merges.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
