Github user iyerr3 commented on a diff in the pull request:
https://github.com/apache/madlib/pull/206#discussion_r154422395
--- Diff: src/ports/postgres/modules/stats/correlation.py_in ---
@@ -165,8 +165,8 @@ def _populate_output_table(schema_madlib, source_table,
output_table,
with MinWarning("info" if verbose else "error"):
start = time()
col_len = len(col_names)
- col_names_as_float_array = py_list_to_sql_string(col_names,
"float8")
col_names_as_text_array = py_list_to_sql_string(col_names,
"varchar")
+ col_names_as_float_array = py_list_to_sql_string(col_names,
"float8")
--- End diff --
Don't need this variable anymore
---
