[ 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 wrong 2. Throw an error if you sort on a non-indexed field (SOLR-9) I still think it should have a default sort order (SQL does) but as long as it does not throw an ArrayIndexOutOfBoundsException exception I am happy :) What should happen with the 'legacy' format? a. leave it as is. (perhaps add a comment for the next person who can't figure out what it does) b. get rid of it. > Allow default sort order > ------------------------ > > Key: SOLR-99 > URL: https://issues.apache.org/jira/browse/SOLR-99 > Project: Solr > Issue Type: Improvement > Components: search > Affects Versions: 1.2 > Reporter: Ryan McKinley > Priority: Minor > Fix For: 1.2 > > Attachments: DefaultSortOrder.patch, DefaultSortOrder.patch > > > The current search throws an "ArrayIndexOutOfBoundsException" if you specify > a sort field but do not include an order. This is anoying and difficult to > debug (when you are starting) > Here is a patch to avoid the exception and use the default sort order if you > only specify a field. I'm not sure the 'null' case was even possible in the > current code: > Index: QueryParsing.java > =================================================================== > --- QueryParsing.java (revision 494681) > +++ QueryParsing.java (working copy) > @@ -186,13 +186,12 @@ > } > > // get the direction of the sort > - str=parts[pos]; > - if ("top".equals(str) || "desc".equals(str)) { > - top=true; > - } else if ("bottom".equals(str) || "asc".equals(str)) { > - top=false; > - } else { > - return null; // must not be a sort command > + // by default, top is true, only change it if it is "bottom" or "asc" > + if( parts.length > pos ) { > + str=parts[pos]; > + if ("bottom".equals(str) || "asc".equals(str)) { > + top=false; > + } > } -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: https://issues.apache.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira