subject:"\[GitHub\] \[spark\] zhengruifeng commented on issue #26858\: \[SPARK\-30120\]\[ML\] Use BoundedPriorityQueue for small dataset in LSH approxNearestNeighbors"

[GitHub] [spark] zhengruifeng commented on issue #26858: [SPARK-30120][ML] Use BoundedPriorityQueue for small dataset in LSH approxNearestNeighbors

2019-12-24 Thread GitBox

zhengruifeng commented on issue #26858: [SPARK-30120][ML] Use BoundedPriorityQueue for small dataset in LSH approxNearestNeighbors URL: https://github.com/apache/spark/pull/26858#issuecomment-568829004 I guess we do not need BoundedPriorityQueue any more, and maybe OK to close this PR?

[GitHub] [spark] zhengruifeng commented on issue #26858: [SPARK-30120][ML] Use BoundedPriorityQueue for small dataset in LSH approxNearestNeighbors

2019-12-12 Thread GitBox

zhengruifeng commented on issue #26858: [SPARK-30120][ML] Use BoundedPriorityQueue for small dataset in LSH approxNearestNeighbors URL: https://github.com/apache/spark/pull/26858#issuecomment-564916604 I am afraid this PR is wrong. > Use BoundedPriorityQueue for small dataset in LSH