[ 
https://issues.apache.org/jira/browse/SOLR-170?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Ryan McKinley updated SOLR-170:
-------------------------------

    Attachment: SOLR-170-sort-parameter.patch

This simple patch adds 'sort' param to StandardRequestHandler and updates the 
tutorial docs to use the sort parameter rather then the ';' syntax

Two things to note:

1. I had to delete some "sort", "true" parameters from the 
BasicFunctionalityTest.java TestSimpleFacetCounts.  "sort=true" does not 
evaluate to a valid sort string and these params were being ignored before

2. This patch includes dummy defined constants in SolrParams.  The trunk branch 
seems to be missing some HIGHLIGHT params.  to get things to compile, i added 
the following - they should obviously not be commited:

  public static final String HIGHLIGHT_FIELDS = null;
  public static final String HIGHLIGHT = null;
  public static final String MAX_SNIPPETS = null;
  public static final String HIGHLIGHT_FORMATTER_CLASS = null;



> StandardRequestHandler should support the 'sort' parameter
> ----------------------------------------------------------
>
>                 Key: SOLR-170
>                 URL: https://issues.apache.org/jira/browse/SOLR-170
>             Project: Solr
>          Issue Type: Improvement
>            Reporter: Ryan McKinley
>            Priority: Minor
>         Attachments: SOLR-170-sort-parameter.patch
>
>
> dismax and standard query handlers should both use the 'sort' parameter.  We 
> should discourage future use of the q=query;sort syntax for 
> StandardRequestHandler 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to