Repository: incubator-madlib
Updated Branches:
  refs/heads/master a2f474007 -> dc0a88ba4


Bugfix: Handle WCC output table error case

If the output table name is NULL, the code doesn't catch and error
out, but errors out because python cannot call strip() function on
NULL. This commit fixes that input parameter (for output table name)
error case scenario.

Closes #160


Project: http://git-wip-us.apache.org/repos/asf/incubator-madlib/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-madlib/commit/dc0a88ba
Tree: http://git-wip-us.apache.org/repos/asf/incubator-madlib/tree/dc0a88ba
Diff: http://git-wip-us.apache.org/repos/asf/incubator-madlib/diff/dc0a88ba

Branch: refs/heads/master
Commit: dc0a88ba4fbd0327ab99638ddf79f04d0b18f94f
Parents: a2f4740
Author: Nandish Jayaram <njaya...@apache.org>
Authored: Tue Aug 1 17:17:47 2017 -0700
Committer: Orhan Kislal <okis...@pivotal.io>
Committed: Thu Aug 3 11:54:02 2017 -0700

----------------------------------------------------------------------
 src/ports/postgres/modules/graph/wcc.py_in | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-madlib/blob/dc0a88ba/src/ports/postgres/modules/graph/wcc.py_in
----------------------------------------------------------------------
diff --git a/src/ports/postgres/modules/graph/wcc.py_in 
b/src/ports/postgres/modules/graph/wcc.py_in
index bf905c8..53ebda9 100644
--- a/src/ports/postgres/modules/graph/wcc.py_in
+++ b/src/ports/postgres/modules/graph/wcc.py_in
@@ -100,8 +100,10 @@ def wcc(schema_madlib, vertex_table, vertex_id, 
edge_table, edge_args,
         v_st = vertex_id
     if not grouping_cols:
         grouping_cols = ''
-    out_table_summary = add_postfix(out_table, "_summary")
 
+    out_table_summary = ''
+    if out_table:
+        out_table_summary = add_postfix(out_table, "_summary")
     grouping_cols_list = split_quoted_delimited_str(grouping_cols)
     validate_wcc_args(schema_madlib, vertex_table, vertex_id, edge_table,
                       edge_params, out_table, out_table_summary,

Reply via email to