Skip to content

Commit

Permalink
Nop out group_by() in SphinxTolerantElastic.
Browse files Browse the repository at this point in the history
ES doesn't support a straightforward group_by analogue. So, in ES, we're mashing subdocuments into the superdocument, making group_by unnecessary.
  • Loading branch information
erikrose committed Dec 7, 2011
1 parent 5be8815 commit 8aacfbe
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions oedipus/__init__.py
Expand Up @@ -660,6 +660,10 @@ def order_by(self, *fields):
return super(SphinxTolerantElastic, self).order_by(
*[transforms.get(f, f) for f in fields])

def group_by(self, *args, **kwargs):
# I don't know if this should really be a no-op or not, for API sanity.
return self


def _check_weights(weights):
"""Verifies weight values are in the appropriate range.
Expand Down

0 comments on commit 8aacfbe

Please sign in to comment.