[
https://issues.apache.org/jira/browse/LUCENE-7361?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15355204#comment-15355204
]
Robert Muir commented on LUCENE-7361:
-------------------------------------
No, I am against this. Its only a matter of time before a debugging message in
infostream becomes a horrible performance bug or similar. sorry.
MemoryIndex is a 1 document toy, it can do whatever it wants.
> Terms.toStringDebug
> -------------------
>
> Key: LUCENE-7361
> URL: https://issues.apache.org/jira/browse/LUCENE-7361
> Project: Lucene - Core
> Issue Type: Improvement
> Reporter: David Smiley
> Attachments: MemoryIndexToString.java
>
>
> While fixing LUCENE-7340, MemoryIndex.toString(), I thought MemoryIndex
> shouldn't need it's own debug toString() impl for its Terms when there could
> be a generic one. So here I propose that we create a
> Terms.toStringDebug(Appendable result, int charLimit, String indent) or
> some-such but probably not override toString() for obvious reasons. Maybe
> also have this on Fields() that simply loops and calls out to the one on
> Terms.
> The format is debatable.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
