[
https://issues.apache.org/jira/browse/SOLR-5746?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14069345#comment-14069345
]
Hoss Man commented on SOLR-5746:
bq. My only review comment would be to prefer using
[
https://issues.apache.org/jira/browse/SOLR-5746?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14069422#comment-14069422
]
ASF subversion and git services commented on SOLR-5746:
---
Commit
[
https://issues.apache.org/jira/browse/SOLR-5746?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14069519#comment-14069519
]
ASF subversion and git services commented on SOLR-5746:
---
Commit
[
https://issues.apache.org/jira/browse/SOLR-5746?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14069524#comment-14069524
]
Hoss Man commented on SOLR-5746:
bq. ...the assertTrue's actually do have messages on
[
https://issues.apache.org/jira/browse/SOLR-5746?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14063425#comment-14063425
]
Maciej Zasada commented on SOLR-5746:
-
Hi [~hossman],
thanks for the code review and
[
https://issues.apache.org/jira/browse/SOLR-5746?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14063623#comment-14063623
]
Alan Woodward commented on SOLR-5746:
-
+1, some nice cleanups here.
My only review
[
https://issues.apache.org/jira/browse/SOLR-5746?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14059771#comment-14059771
]
Maciej Zasada commented on SOLR-5746:
-
bq. 1) can you explain why the need for the new
[
https://issues.apache.org/jira/browse/SOLR-5746?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14059805#comment-14059805
]
Jack Krupansky commented on SOLR-5746:
--
Will the changes for this issue result in a
[
https://issues.apache.org/jira/browse/SOLR-5746?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14059550#comment-14059550
]
Hoss Man commented on SOLR-5746:
Hi Maciej,
I glance over the patch a bit more today:
1)
[
https://issues.apache.org/jira/browse/SOLR-5746?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14057675#comment-14057675
]
Hoss Man commented on SOLR-5746:
Maciej:
At first glance this looks awsome -- i'll try to
[
https://issues.apache.org/jira/browse/SOLR-5746?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13904881#comment-13904881
]
Hoss Man commented on SOLR-5746:
I think what we should do is...
* change the parsing
11 matches
Mail list logo