Uploaded image for project: 'MariaDB Server'
  1. MariaDB Server
  2. MDEV-5477

[PATCH] sphinxSE GROUP BY on multiple attributes

        Details

        • Type: Bug
        • Status: Open
        • Priority: Major
        • Resolution: Unresolved
        • Affects Version/s: 10.0.6
        • Fix Version/s: 10.0
        • Component/s: None
        • Labels:
        • Environment:
          Linux, SphinxSE

          Description

          It is possible to group by on multiple attribute values with the native API and SphinxQL but not with the Storage Engine.

          The patch implements a new mode ("multi:") for the groupby function SPH_GROUPBY_MULTIPLE.

          After the patch it is possible to group by on multiple attribute values with this query:

          SELECT _sph_count, attribute1, attribute2 FROM sphinx_index_table WHERE query = ';groupby=multi:attribute1, attribute2';

            Gliffy Diagrams

              Attachments

                Issue Links

                is blocked by

                Task - A task that needs to be done. MDEV-5480 upgrade sphinxse

                • Major - Major loss of function.
                • Closed
                links to

                Web Link same issue in the sphinxse bug tracker

                  Activity

                  Ascending order - Click to sort in descending order
                  Hide
                  Permalink
                  alex Alex Luft added a comment -

                  Is anything else blocking this issue? Grouping on multiple attributes is really essential for statistical analysis... We use this patch already, works like a charm

                  Show
                  alex Alex Luft added a comment - Is anything else blocking this issue? Grouping on multiple attributes is really essential for statistical analysis... We use this patch already, works like a charm
                  Hide
                  Permalink
                  serg Sergei Golubchik added a comment - - edited

                  Nothing is really blocking it, besides the fact that it's an upstream issue and we generally prefer upstream issues to be fixed upstream. But I see that this was not reported upstream yet.

                  I've now reported it on the SphinxSE bug tracker, see the link above.

                  Let's see how it'll go. We still might end up applying the patch only in MariaDB, although, of course, it would be better if sphinx developers would've taken it.

                  Show
                  serg Sergei Golubchik added a comment - - edited Nothing is really blocking it, besides the fact that it's an upstream issue and we generally prefer upstream issues to be fixed upstream. But I see that this was not reported upstream yet. I've now reported it on the SphinxSE bug tracker, see the link above. Let's see how it'll go. We still might end up applying the patch only in MariaDB, although, of course, it would be better if sphinx developers would've taken it.
                  Hide
                  Permalink
                  elenst Elena Stepanova added a comment -

                  The upstream issue hasn't had much activity so far, should we maybe apply the patch?

                  Show
                  elenst Elena Stepanova added a comment - The upstream issue hasn't had much activity so far, should we maybe apply the patch?
                  Hide
                  Permalink
                  alex Alex Luft added a comment -

                  I would really appreciate it... I also implemented some features from VER_COMMAND_SEARCH 0x11B (http://sphinxsearch.com/bugs/view.php?id=2007) but there is not much activity on sphinxSE.

                  Show
                  alex Alex Luft added a comment - I would really appreciate it... I also implemented some features from VER_COMMAND_SEARCH 0x11B ( http://sphinxsearch.com/bugs/view.php?id=2007 ) but there is not much activity on sphinxSE.

                    People

                    • Assignee:
                      Unassigned
                      Reporter:
                      alex Alex Luft
                    • Votes:
                      0 Vote for this issue
                      Watchers:
                      3 Start watching this issue

                      Dates

                      • Created:
                        Updated: