Github user kaknikhil commented on a diff in the pull request:
https://github.com/apache/madlib/pull/195#discussion_r151226056
--- Diff: src/ports/postgres/modules/utilities/utilities.py_in ---
@@ -709,16 +709,35 @@ def _check_groups(tbl1, tbl2, grp_list):
return ' AND '.join([" {tbl1}.{i} = {tbl2}.{i} ".format(**locals())
for i in grp_list])
-
-def _grp_from_table(tbl, grp_list):
- """
- Helper function for selecting grouping columns of a table
+def get_filtered_cols_subquery_str(include_from_table, exclude_from_table,
--- End diff --
we removed the _ because get_filtered_cols_subquery_str is not for internal
use, it is called by pagerank and the python files.
---
