So, I definitely think this should all be part of TS proper - I'd love for Thinking Sphinx to opt for smarter string facets when possible (as it removes the need for the class_crc attribute, and tracking which models are being indexed, which has been a pain from the beginning).
And same for string facets - although we're still faced with the existing limitation there when having arrays of strings - MVAs are still integer-only.
Also, 1.10-beta has a useful option of sql_field_string - declaring both a field and attribute from a single column. This is a natural fit for :sortable => true.
And the newly released 2.0.1-beta improves things further with string attributes - they can be used for sorting and grouping - wish they'd hurry up with the filtering as well! (I've had a brief attempt at getting TS working with Sphinx 2.0.1-beta - Sphinx was having problems starting up via Ruby code, but was fine if I called it from the command line. Will investigate more soon).
The trick will be having it all degrade nicely for older versions of Sphinx. There's already a check or two in Riddle for this - Riddle.loaded_version. If you want to take a stab at it, that'd be wonderful - but otherwise, I'm guessing you'll be in Berlin for Euruko? Perhaps can work on it together then?
--
Pat
> --
> You received this message because you are subscribed to the Google Groups "Thinking Sphinx" group.
> To post to this group, send email to thinkin...@googlegroups.com.
> To unsubscribe from this group, send email to thinking-sphi...@googlegroups.com.
> For more options, visit this group at http://groups.google.com/group/thinking-sphinx?hl=en.
>
thanks for the feedback!
On May 3, 2011, at 2:41 PM, Pat Allan wrote:
> Okay, here's a few thoughts on the matter - finally (sorry for the delay).
>
> So, I definitely think this should all be part of TS proper - I'd love for Thinking Sphinx to opt for smarter string facets when possible (as it removes the need for the class_crc attribute, and tracking which models are being indexed, which has been a pain from the beginning).
Yes, that would be awesome!
> And same for string facets - although we're still faced with the existing limitation there when having arrays of strings - MVAs are still integer-only.
I haven't even thought about the MVA issue ... Still, maybe it's possible to simplify the lookup code (right now it's quite expensive). Or maybe it makes sense to just not support this directly? Is it much used anyway? While taking a feature out always sucks, it might benefit the overall codebase.
> Also, 1.10-beta has a useful option of sql_field_string - declaring both a field and attribute from a single column. This is a natural fit for :sortable => true.
+1
> And the newly released 2.0.1-beta improves things further with string attributes - they can be used for sorting and grouping - wish they'd hurry up with the filtering as well! (I've had a brief attempt at getting TS working with Sphinx 2.0.1-beta - Sphinx was having problems starting up via Ruby code, but was fine if I called it from the command line. Will investigate more soon).
Gosh, that would be *so* great. However, in terms of facets, sorting and grouping would actually be enough. You could then – in contrast to my quick and dirty implementation for 1.10-beta – even sort by group rather than just count.
> The trick will be having it all degrade nicely for older versions of Sphinx. There's already a check or two in Riddle for this - Riddle.loaded_version. If you want to take a stab at it, that'd be wonderful - but otherwise, I'm guessing you'll be in Berlin for Euruko? Perhaps can work on it together then?
Sure, I'll be there for a couple of days. I'm assuming you'll stay a while, too? Sounds like a good fit for a pairing session at co_up or Euruko hallway ...
- C.