[
https://issues.apache.org/jira/browse/SOLR-99?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12464053
]
Ryan McKinley commented on SOLR-99:
---
oops, by SOLR-6, i really mean SOLR-9
I appologize in advance for my dyslexia.
[
https://issues.apache.org/jira/browse/SOLR-99?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12464052
]
Ryan McKinley commented on SOLR-99:
---
I just attached a new version. It throws an exception when:
1) missing sort orde
[
https://issues.apache.org/jira/browse/SOLR-99?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12464013
]
Hoss Man commented on SOLR-99:
--
the problem with a default sort order (which ANSI SQL does wrong in my opinion)
is that it
[
https://issues.apache.org/jira/browse/SOLR-99?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12463813
]
Ryan McKinley commented on SOLR-99:
---
Ok, we all agree, that it should
1. Throw a reasonable error if something goes
[
https://issues.apache.org/jira/browse/SOLR-99?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12463740
]
Yonik Seeley commented on SOLR-99:
--
I agree that throwing a ParseException (ideally a SolrException with a code of
400
[
https://issues.apache.org/jira/browse/SOLR-99?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12463702
]
Yonik Seeley commented on SOLR-99:
--
The number was there because I originally thought about allowing a chain of
sorts
[
https://issues.apache.org/jira/browse/SOLR-99?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12463695
]
Hoss Man commented on SOLR-99:
--
specifying the number of results to return as part of the sort information is
in fact some
[
https://issues.apache.org/jira/browse/SOLR-99?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12463530
]
Ryan McKinley commented on SOLR-99:
---
I apologize... the first version does not pass
ConvertedLegacyTest.testABunchOfC