Skip to content

Commit

Permalink
[bugfix] Fix Expression.show() with a key field that isn't the first key
Browse files Browse the repository at this point in the history
Fixes hail-is#5449

We don't have machinery for testing performance behavior of something
like show() right now, so I can't test it easily. But I did verify by
hand.
  • Loading branch information
tpoterba committed Feb 27, 2019
1 parent dcc7b86 commit dd3fcb6
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion hail/python/hail/expr/expressions/base_expression.py
Expand Up @@ -706,7 +706,7 @@ def _show(self, n, width, truncate, types):
name = source._fields_inverse.get(self, name)
t = self._to_table(name)
if name in t.key:
t = t.key_by(name).select()
t = t.order_by(*t.key).select(name)
return t._show(n, width, truncate, types)


Expand Down

0 comments on commit dd3fcb6

Please sign in to comment.