store only sort column indexes in logical OrderBy operator

uwescience · Aug 28, 2017 · c6fdede · c6fdede
1 parent fe87d73
commit c6fdede
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/raco/backends/myria/myria.py b/raco/backends/myria/myria.py
@@ -601,7 +601,7 @@ def compileme(self, inputsym):
         return {
             "opType": "InMemoryOrderBy",
             "argChild": inputsym,
-            "argSortColumns": [col.position for col in self.sort_columns],
+            "argSortColumns": self.sort_columns,
             "argAscending": self.ascending
         }
 

diff --git a/raco/myrial/interpreter.py b/raco/myrial/interpreter.py
@@ -258,10 +258,10 @@ def bagcomp(self, from_clause, where_clause, emit_clause,
                 raise InvalidStatementException(
                     "An ORDER BY clause must be accompanied by a LIMIT clause")
             orderby_cols, orderby_ords = zip(*orderby_clause)
-            orderby_cols = [get_unnamed_ref(col, op.scheme())
+            orderby_idxs = [get_unnamed_ref(col, op.scheme()).position
                             for col in orderby_cols]
             op = raco.algebra.OrderBy(input=op,
-                                      sort_columns=orderby_cols,
+                                      sort_columns=orderby_idxs,
                                       ascending=orderby_ords)
 
         if limit_clause: