You are viewing a plain text version of this content. The canonical link for it is here.
Posted to solr-dev@lucene.apache.org by "Ryan McKinley (JIRA)" <ji...@apache.org> on 2007/02/22 08:38:05 UTC

[jira] Updated: (SOLR-170) StandardRequestHandler should support the 'sort' parameter

     [ 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.