[GitHub] madlib pull request #195: Feature: Add grouping support to HITS

2017-11-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/madlib/pull/195 ---

[GitHub] madlib pull request #195: Feature: Add grouping support to HITS

2017-11-20 Thread iyerr3
Github user iyerr3 commented on a diff in the pull request: https://github.com/apache/madlib/pull/195#discussion_r152136163 --- Diff: src/ports/postgres/modules/utilities/utilities.py_in --- @@ -709,16 +709,35 @@ def _check_groups(tbl1, tbl2, grp_list): return ' AND

[GitHub] madlib pull request #195: Feature: Add grouping support to HITS

2017-11-16 Thread iyerr3
Github user iyerr3 commented on a diff in the pull request: https://github.com/apache/madlib/pull/195#discussion_r151526779 --- Diff: src/ports/postgres/modules/graph/hits.py_in --- @@ -95,234 +109,391 @@ def hits(schema_madlib, vertex_table, vertex_id, edge_table, edge_args,

[GitHub] madlib pull request #195: Feature: Add grouping support to HITS

2017-11-15 Thread kaknikhil
Github user kaknikhil commented on a diff in the pull request: https://github.com/apache/madlib/pull/195#discussion_r151226712 --- Diff: src/ports/postgres/modules/graph/hits.py_in --- @@ -95,234 +109,391 @@ def hits(schema_madlib, vertex_table, vertex_id, edge_table, edge_args,

[GitHub] madlib pull request #195: Feature: Add grouping support to HITS

2017-11-15 Thread kaknikhil
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

[GitHub] madlib pull request #195: Feature: Add grouping support to HITS

2017-11-15 Thread kaknikhil
Github user kaknikhil commented on a diff in the pull request: https://github.com/apache/madlib/pull/195#discussion_r151225853 --- Diff: src/ports/postgres/modules/graph/hits.sql_in --- @@ -164,10 +168,20 @@ INSERT INTO edge VALUES (3, 0, 1), (4, 0, 1), (5, 6, 1),

[GitHub] madlib pull request #195: Feature: Add grouping support to HITS

2017-11-15 Thread jingyimei
Github user jingyimei commented on a diff in the pull request: https://github.com/apache/madlib/pull/195#discussion_r151063100 --- Diff: src/ports/postgres/modules/graph/graph_utils.py_in --- @@ -109,6 +110,85 @@ def validate_graph_coding(vertex_table, vertex_id, edge_table,

[GitHub] madlib pull request #195: Feature: Add grouping support to HITS

2017-11-15 Thread jingyimei
Github user jingyimei commented on a diff in the pull request: https://github.com/apache/madlib/pull/195#discussion_r151064800 --- Diff: src/ports/postgres/modules/graph/hits.py_in --- @@ -95,234 +109,391 @@ def hits(schema_madlib, vertex_table, vertex_id, edge_table, edge_args,

[GitHub] madlib pull request #195: Feature: Add grouping support to HITS

2017-11-15 Thread jingyimei
Github user jingyimei commented on a diff in the pull request: https://github.com/apache/madlib/pull/195#discussion_r151061922 --- Diff: src/ports/postgres/modules/utilities/utilities.py_in --- @@ -709,16 +709,35 @@ def _check_groups(tbl1, tbl2, grp_list): return ' AND

[GitHub] madlib pull request #195: Feature: Add grouping support to HITS

2017-11-13 Thread iyerr3
Github user iyerr3 commented on a diff in the pull request: https://github.com/apache/madlib/pull/195#discussion_r150638553 --- Diff: src/ports/postgres/modules/utilities/validate_args.py_in --- @@ -262,6 +262,13 @@ def get_first_schema(table_name): return None #

[GitHub] madlib pull request #195: Feature: Add grouping support to HITS

2017-11-13 Thread iyerr3
Github user iyerr3 commented on a diff in the pull request: https://github.com/apache/madlib/pull/195#discussion_r150637914 --- Diff: src/ports/postgres/modules/utilities/utilities.py_in --- @@ -709,6 +709,17 @@ def _check_groups(tbl1, tbl2, grp_list): return ' AND

[GitHub] madlib pull request #195: Feature: Add grouping support to HITS

2017-11-13 Thread orhankislal
Github user orhankislal commented on a diff in the pull request: https://github.com/apache/madlib/pull/195#discussion_r150629090 --- Diff: src/ports/postgres/modules/graph/graph_utils.py_in --- @@ -109,6 +110,85 @@ def validate_graph_coding(vertex_table, vertex_id, edge_table,

[GitHub] madlib pull request #195: Feature: Add grouping support to HITS

2017-11-13 Thread orhankislal
Github user orhankislal commented on a diff in the pull request: https://github.com/apache/madlib/pull/195#discussion_r150629575 --- Diff: src/ports/postgres/modules/graph/graph_utils.py_in --- @@ -109,6 +110,85 @@ def validate_graph_coding(vertex_table, vertex_id, edge_table,

[GitHub] madlib pull request #195: Feature: Add grouping support to HITS

2017-11-06 Thread kaknikhil
GitHub user kaknikhil opened a pull request: https://github.com/apache/madlib/pull/195 Feature: Add grouping support to HITS JIRA: MADLIB-1151 Changes to support grouping column in HITS. Update queries to use group by and other necessary sql constructs. You can merge