Github user reductionista commented on a diff in the pull request:
https://github.com/apache/madlib/pull/309#discussion_r209807557
--- Diff:
src/ports/postgres/modules/elastic_net/elastic_net_generate_result.py_in ---
@@ -34,26 +36,30 @@ def _elastic_net_generate_result(optimizer,
iteration_run, **args):
col_grp_key = args['col_grp_key']
grouping_str = args['grouping_str']
cols_types = dict(get_cols_and_types(args["tbl_source"]))
- grouping_str1 = grouping_column + ","
+ grouping_cols_list = split_quoted_delimited_str(grouping_column)
+ grouping_str1 = ','.join(['{0} AS {1}'.format(g, quote_ident(g))
+ for g in grouping_cols_list])
--- End diff --
Nice! Would not have thought of this approach.
---
