[GitHub] madlib pull request #279: KNN: make install check asserts deterministic

[kaknikhil <gi...@git.apache.org>]

GitHub user kaknikhil opened a pull request:

    https://github.com/apache/madlib/pull/279

    KNN: make install check asserts deterministic

    knn install check had a couple of array_agg asserts which were not
    deterministic. This commit adds order by to the array agg function to
    make it deterministic.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/madlib/madlib bugfix/knn_ic

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/madlib/pull/279.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #279
    
----
commit c117afa614bb84c4cc36b466934247e51d2eff4e
Author: Nikhil Kak <nk...@...>
Date:   2018-06-14T17:20:55Z

    KNN: make install check asserts deterministic
    
    knn install check had a couple of array_agg asserts which were not
    deterministic. This commit adds order by to the array agg function to
    make it derterministic.

----


---

[GitHub] madlib pull request #279: KNN: make install check asserts deterministic

[asfgit <gi...@git.apache.org>]

Github user asfgit closed the pull request at:

    https://github.com/apache/madlib/pull/279


---

[GitHub] madlib issue #279: KNN: make install check asserts deterministic

[asfgit <gi...@git.apache.org>]

Github user asfgit commented on the issue:

    https://github.com/apache/madlib/pull/279
  
    
    Refer to this link for build results (access rights to CI server needed): 
    https://builds.apache.org/job/madlib-pr-build/515/



---