[ https://issues.apache.org/jira/browse/LUCENE-3756?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13202502#comment-13202502 ]
Uwe Schindler commented on LUCENE-3756: --------------------------------------- Shai: Thanks for the comment, I have no idea how to fight against this anti-builder mentality. There is nothing that forces you to use chaining. See also LUCENE-3736, I brought all arguments. > Don't allow IndexWriterConfig setters to chain > ---------------------------------------------- > > Key: LUCENE-3756 > URL: https://issues.apache.org/jira/browse/LUCENE-3756 > Project: Lucene - Java > Issue Type: Improvement > Reporter: Michael McCandless > Assignee: Michael McCandless > > Spinoff from LUCENE-3736. > I don't like that IndexWriterConfig's setters are chainable; it > results in code in our tests like this: > {noformat} > IndexWriter writer = new IndexWriter(dir, newIndexWriterConfig( > TEST_VERSION_CURRENT, new > MockAnalyzer(random)).setMaxBufferedDocs(2).setMergePolicy(newLogMergePolicy())); > {noformat} > I think in general we should avoid chaining since it encourages hard > to read code (code is already hard enough to read!). -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa 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