[GitHub] madlib pull request #330: Margins: Copy summary table instead of renaming

2018-10-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/madlib/pull/330 ---

[GitHub] madlib pull request #330: Margins: Copy summary table instead of renaming

2018-10-03 Thread kaknikhil
Github user kaknikhil commented on a diff in the pull request: https://github.com/apache/madlib/pull/330#discussion_r222415826 --- Diff: src/ports/postgres/modules/regress/marginal.py_in --- @@ -522,9 +520,8 @@ def margins_mlogregr(schema_madlib, source_table, out_table,

[GitHub] madlib pull request #330: Margins: Copy summary table instead of renaming

2018-10-03 Thread kaknikhil
Github user kaknikhil commented on a diff in the pull request: https://github.com/apache/madlib/pull/330#discussion_r222418291 --- Diff: src/ports/postgres/modules/regress/marginal.py_in --- @@ -522,9 +520,8 @@ def margins_mlogregr(schema_madlib, source_table, out_table,

[GitHub] madlib pull request #330: Margins: Copy summary table instead of renaming

2018-10-03 Thread orhankislal
GitHub user orhankislal opened a pull request: https://github.com/apache/madlib/pull/330 Margins: Copy summary table instead of renaming JIRA: MADLIB-1274 Margins summary table gets dropped since its schema remains pg_temp. This commit fixed the issue by copying the cont