[jira] [Commented] (LUCENE-8248) Rename MergePolicyWrapper to FilterMergePolicy and override all of MergePolicy

2018-04-14 Thread Lucene/Solr QA (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16438587#comment-16438587 ] Lucene/Solr QA commented on LUCENE-8248: | (x) *{color:red}-1 overall{color}* | \

Re: [jira] [Commented] (LUCENE-8248) Rename MergePolicyWrapper to FilterMergePolicy and override all of MergePolicy

2018-04-13 Thread Michael Sokolov
yes, thanks! On Fri, Apr 13, 2018 at 7:05 PM, Michael McCandless (JIRA) wrote: > > [ https://issues.apache.org/jira/browse/LUCENE-8248?page= > com.atlassian.jira.plugin.system.issuetabpanels:comment- > tabpanel&focusedCommentId=16438060#comment-16438060 ] > > Michael McCandless commented on

[jira] [Commented] (LUCENE-8248) Rename MergePolicyWrapper to FilterMergePolicy and override all of MergePolicy

2018-04-13 Thread Michael McCandless (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16438060#comment-16438060 ] Michael McCandless commented on LUCENE-8248: Woops, thanks [~varunthacker]!

[jira] [Commented] (LUCENE-8248) Rename MergePolicyWrapper to FilterMergePolicy and override all of MergePolicy

2018-04-13 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16438043#comment-16438043 ] ASF subversion and git services commented on LUCENE-8248: - Commit

[jira] [Commented] (LUCENE-8248) Rename MergePolicyWrapper to FilterMergePolicy and override all of MergePolicy

2018-04-13 Thread Varun Thacker (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16438031#comment-16438031 ] Varun Thacker commented on LUCENE-8248: --- I'll fix the precommit issue by removing t

[jira] [Commented] (LUCENE-8248) Rename MergePolicyWrapper to FilterMergePolicy and override all of MergePolicy

2018-04-13 Thread Mike Sokolov (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16437977#comment-16437977 ] Mike Sokolov commented on LUCENE-8248: -- yw - thanks for submitting [~mikemccand] >

[jira] [Commented] (LUCENE-8248) Rename MergePolicyWrapper to FilterMergePolicy and override all of MergePolicy

2018-04-13 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16437873#comment-16437873 ] ASF subversion and git services commented on LUCENE-8248: - Commit

[jira] [Commented] (LUCENE-8248) Rename MergePolicyWrapper to FilterMergePolicy and override all of MergePolicy

2018-04-13 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16437864#comment-16437864 ] ASF subversion and git services commented on LUCENE-8248: - Commit

[jira] [Commented] (LUCENE-8248) Rename MergePolicyWrapper to FilterMergePolicy and override all of MergePolicy

2018-04-13 Thread David Smiley (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16437830#comment-16437830 ] David Smiley commented on LUCENE-8248: -- Thanks Mike Sokolov; the inconsistent name w

[jira] [Commented] (LUCENE-8248) Rename MergePolicyWrapper to FilterMergePolicy and override all of MergePolicy

2018-04-13 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16437829#comment-16437829 ] ASF subversion and git services commented on LUCENE-8248: - Commit

[jira] [Commented] (LUCENE-8248) Rename MergePolicyWrapper to FilterMergePolicy and override all of MergePolicy

2018-04-13 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16437806#comment-16437806 ] ASF subversion and git services commented on LUCENE-8248: - Commit

[jira] [Commented] (LUCENE-8248) Rename MergePolicyWrapper to FilterMergePolicy and override all of MergePolicy

2018-04-13 Thread Michael McCandless (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16437766#comment-16437766 ] Michael McCandless commented on LUCENE-8248: Thanks [~msoko...@gmail.com]; ne

[jira] [Commented] (LUCENE-8248) Rename MergePolicyWrapper to FilterMergePolicy and override all of MergePolicy

2018-04-13 Thread Mike Sokolov (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16437745#comment-16437745 ] Mike Sokolov commented on LUCENE-8248: -- I think this build failed due to some files

[jira] [Commented] (LUCENE-8248) Rename MergePolicyWrapper to FilterMergePolicy and override all of MergePolicy

2018-04-12 Thread Lucene/Solr QA (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16436716#comment-16436716 ] Lucene/Solr QA commented on LUCENE-8248: | (x) *{color:red}-1 overall{color}* | \

[jira] [Commented] (LUCENE-8248) Rename MergePolicyWrapper to FilterMergePolicy and override all of MergePolicy

2018-04-12 Thread Mike Sokolov (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16436335#comment-16436335 ] Mike Sokolov commented on LUCENE-8248: -- OK, I restored {{TestNoMergePolicy.testMetho

[jira] [Commented] (LUCENE-8248) Rename MergePolicyWrapper to FilterMergePolicy and override all of MergePolicy

2018-04-12 Thread Michael McCandless (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16435933#comment-16435933 ] Michael McCandless commented on LUCENE-8248: I suspect there was a time when

[jira] [Commented] (LUCENE-8248) Rename MergePolicyWrapper to FilterMergePolicy and override all of MergePolicy

2018-04-11 Thread Mike Sokolov (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16434509#comment-16434509 ] Mike Sokolov commented on LUCENE-8248: -- I guess I'm confused why NoMergePolicy must

[jira] [Commented] (LUCENE-8248) Rename MergePolicyWrapper to FilterMergePolicy and override all of MergePolicy

2018-04-11 Thread Mike Sokolov (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16434493#comment-16434493 ] Mike Sokolov commented on LUCENE-8248: -- Oh! Maybe it *should* extend `FilterMergePol

[jira] [Commented] (LUCENE-8248) Rename MergePolicyWrapper to FilterMergePolicy and override all of MergePolicy

2018-04-11 Thread Michael McCandless (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16434208#comment-16434208 ] Michael McCandless commented on LUCENE-8248: Thanks [~msoko...@gmail.com], ne

[jira] [Commented] (LUCENE-8248) Rename MergePolicyWrapper to FilterMergePolicy and override all of MergePolicy

2018-04-11 Thread Mike Sokolov (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16434082#comment-16434082 ] Mike Sokolov commented on LUCENE-8248: -- OK there was already a {{TestMergePolicyWrap

[jira] [Commented] (LUCENE-8248) Rename MergePolicyWrapper to FilterMergePolicy and override all of MergePolicy

2018-04-11 Thread Mike Sokolov (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16433994#comment-16433994 ] Mike Sokolov commented on LUCENE-8248: -- Oh yes that makes sense. Then we are enforci