I've been debugging the code and found the culprit:

class Riddle::Query::Select
...
  def to_sql
    sql = "SELECT #{ extended_values } FROM #{ @indices.join(', ') }"
    sql << " WHERE #{ combined_wheres }" if wheres?
    sql << " #{group_prefix} #{escape_column(@group_by)}" if !@group_by.nil?
    unless @order_within_group_by.nil?
      sql << " WITHIN GROUP ORDER BY
#{escape_columns(@order_within_group_by)}"
    end
    sql << " HAVING #{@having.join(' AND ')}" unless @having.empty?
    sql << " ORDER BY #{escape_columns(@order_by)}" if !@order_by.nil?
    sql << " #{limit_clause}"   unless @limit.nil? && @offset.nil?
    sql << " #{options_clause}" unless @options.empty?

    sql
  end

It calls escape_column for @group_by, instead of escape_columns as for
@order_within_group_by.




On Wed, Feb 26, 2014 at 1:36 PM, J. Garcia <[email protected]> wrote:

> Pat,
> Thanks for your response.
> Just passing any of the two fields in a string worked fine. When I tried
> to pass the two in a comma-separated string I got this stacktrace:
>
>
> ThinkingSphinx::SyntaxError: sphinxql: syntax error, unexpected
> $undefined, expecting IDENT (or 5 other tokens)  near '`user_id,` type
> LIMIT 0, 20; SHOW META'
>     from
> .../gems/thinking-sphinx-3.1.0/lib/thinking_sphinx/connection.rb:90:in
> `rescue in query'
>
>
>
>
> On Tue, Feb 25, 2014 at 11:59 PM, Pat Allan <[email protected]>wrote:
>
>> Can you try passing in a string of comma-separated attributes, instead of
>> an array of symbols?
>>
>> --
>> Pat
>>
>> On 26 Feb 2014, at 4:47 am, J. Garcia <[email protected]> wrote:
>>
>> Hi,
>> According to Sphinx, it is possible to do a group_by on multiple fields
>> since version 2.1.2: http://sphinxsearch.com/bugs/view.php?id=1453
>>
>> I haven't found a way to do a search on multiple fields with Thinking
>> Sphinx.
>> Is it possible to group by multiple fields on Thinking Sphinx?
>> I have tried by including several group_by but does not seem to work.
>> And passing an array of attributes does not seem to be accepted by
>> Thinking Sphinx:
>>
>> ThinkingSphinx::SyntaxError: sphinxql: syntax error, unexpected
>> $undefined, expecting IDENT (or 5 other tokens) near '`[:user_id,` :type]
>> LIMIT 0, 20; SHOW META'
>>     from
>> .../gems/thinking-sphinx-3.1.0/lib/thinking_sphinx/connection.rb:90:in
>> `rescue in query'
>>
>> J.
>>
>>
>> --
>> You received this message because you are subscribed to the Google Groups
>> "Thinking Sphinx" group.
>> To unsubscribe from this group and stop receiving emails from it, send an
>> email to [email protected].
>>
>> To post to this group, send email to [email protected].
>> Visit this group at http://groups.google.com/group/thinking-sphinx.
>> For more options, visit https://groups.google.com/groups/opt_out.
>>
>>
>>  --
>> You received this message because you are subscribed to a topic in the
>> Google Groups "Thinking Sphinx" group.
>> To unsubscribe from this topic, visit
>> https://groups.google.com/d/topic/thinking-sphinx/bZpFqr1-iTU/unsubscribe
>> .
>> To unsubscribe from this group and all its topics, send an email to
>> [email protected].
>> To post to this group, send email to [email protected].
>> Visit this group at http://groups.google.com/group/thinking-sphinx.
>> For more options, visit https://groups.google.com/groups/opt_out.
>>
>
>

-- 
You received this message because you are subscribed to the Google Groups 
"Thinking Sphinx" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/thinking-sphinx.
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to