madlib-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From kaknikhil <...@git.apache.org>
Subject [GitHub] madlib pull request #195: Feature: Add grouping support to HITS
Date Wed, 15 Nov 2017 19:20:26 GMT
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),
    -(6, 3, 1);
    +(6, 3, 1),
    +(0, 1, 2),
    +(0, 2, 2),
    +(0, 4, 2),
    +(1, 2, 2),
    +(1, 3, 2),
    +(2, 3, 2),
    +(3, 0, 2),
    +(4, 0, 2),
    +(5, 6, 2),
    +(6, 3, 2);
     </pre>
     
    --# Compute the HITS scores:
    +-# Running HITS with default values for optional parameters:
    --- End diff --
    
    @jingyimei good point. we checked in a commit to remove the extra user_id `2` from the
edge table when hits is called without the grouping column.  


---

Mime
View raw message