[jira] [Commented] (LUCENE-8152) Simplify conditionals in JoinUtil

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

[jira] [Commented] (LUCENE-8152) Simplify conditionals in JoinUtil

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

[jira] [Commented] (LUCENE-8152) Simplify conditionals in JoinUtil

2018-02-12 Thread Steve Rowe (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8152?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16361259#comment-16361259 ] Steve Rowe commented on LUCENE-8152: Starting with the {{aa157f18335}} commit on this

[jira] [Commented] (LUCENE-8152) Simplify conditionals in JoinUtil

2018-02-12 Thread Adrien Grand (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8152?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16361043#comment-16361043 ] Adrien Grand commented on LUCENE-8152: -- I'm afraid this will require rewriting upstr

[jira] [Commented] (LUCENE-8152) Simplify conditionals in JoinUtil

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

[jira] [Commented] (LUCENE-8152) Simplify conditionals in JoinUtil

2018-02-12 Thread Horatiu Lazu (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8152?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16360980#comment-16360980 ] Horatiu Lazu commented on LUCENE-8152: -- Hmm. You can amend the author of the last co

[jira] [Commented] (LUCENE-8152) Simplify conditionals in JoinUtil

2018-02-12 Thread Adrien Grand (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8152?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16360915#comment-16360915 ] Adrien Grand commented on LUCENE-8152: -- Sorry I had not paid attention to the Github

[jira] [Commented] (LUCENE-8152) Simplify conditionals in JoinUtil

2018-02-12 Thread Horatiu Lazu (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8152?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16360861#comment-16360861 ] Horatiu Lazu commented on LUCENE-8152: -- Thanks, is it possible to have accepted my p

[jira] [Commented] (LUCENE-8152) Simplify conditionals in JoinUtil

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

[jira] [Commented] (LUCENE-8152) Simplify conditionals in JoinUtil

2018-02-05 Thread Horatiu Lazu (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8152?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16352738#comment-16352738 ] Horatiu Lazu commented on LUCENE-8152: -- Okay, thanks! > Simplify conditionals in Jo

[jira] [Commented] (LUCENE-8152) Simplify conditionals in JoinUtil

2018-02-05 Thread Adrien Grand (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8152?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16352391#comment-16352391 ] Adrien Grand commented on LUCENE-8152: -- This looks good at first sight. I'll run tes

[jira] [Commented] (LUCENE-8152) Simplify conditionals in JoinUtil

2018-02-02 Thread Horatiu Lazu (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8152?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16351246#comment-16351246 ] Horatiu Lazu commented on LUCENE-8152: -- I added changes throughout the codebase, hop

[jira] [Commented] (LUCENE-8152) Simplify conditionals in JoinUtil

2018-02-02 Thread Adrien Grand (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8152?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16350439#comment-16350439 ] Adrien Grand commented on LUCENE-8152: -- That would be great. > Simplify conditional

[jira] [Commented] (LUCENE-8152) Simplify conditionals in JoinUtil

2018-02-02 Thread Horatiu Lazu (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8152?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16350370#comment-16350370 ] Horatiu Lazu commented on LUCENE-8152: -- Yes, I'll change them tonight. Should I do i

[jira] [Commented] (LUCENE-8152) Simplify conditionals in JoinUtil

2018-02-02 Thread Martijn van Groningen (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8152?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16350317#comment-16350317 ] Martijn van Groningen commented on LUCENE-8152: --- +1 That is much cleaner >

[jira] [Commented] (LUCENE-8152) Simplify conditionals in JoinUtil

2018-02-02 Thread Adrien Grand (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8152?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16350288#comment-16350288 ] Adrien Grand commented on LUCENE-8152: -- Your patch looks good to me. Would you like

[jira] [Commented] (LUCENE-8152) Simplify conditionals in JoinUtil

2018-02-01 Thread Horatiu Lazu (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-8152?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16349753#comment-16349753 ] Horatiu Lazu commented on LUCENE-8152: -- Similar patterns seem to exist in other part