You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flink.apache.org by ch...@apache.org on 2017/01/06 15:43:47 UTC

[3/6] flink git commit: [FLINK-4148] Fix min distance calculation in QuadTree

[FLINK-4148] Fix min distance calculation in QuadTree


Project: http://git-wip-us.apache.org/repos/asf/flink/repo
Commit: http://git-wip-us.apache.org/repos/asf/flink/commit/9d99f2bd
Tree: http://git-wip-us.apache.org/repos/asf/flink/tree/9d99f2bd
Diff: http://git-wip-us.apache.org/repos/asf/flink/diff/9d99f2bd

Branch: refs/heads/release-1.2
Commit: 9d99f2bd4a29b748905e55a774ff04f933b6b00f
Parents: aaf9612
Author: Alexey Diomin <di...@gmail.com>
Authored: Mon Jul 4 19:13:11 2016 +0400
Committer: zentol <ch...@apache.org>
Committed: Fri Jan 6 12:54:46 2017 +0100

----------------------------------------------------------------------
 .../flink-ml/src/main/scala/org/apache/flink/ml/nn/QuadTree.scala  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/flink/blob/9d99f2bd/flink-libraries/flink-ml/src/main/scala/org/apache/flink/ml/nn/QuadTree.scala
----------------------------------------------------------------------
diff --git a/flink-libraries/flink-ml/src/main/scala/org/apache/flink/ml/nn/QuadTree.scala b/flink-libraries/flink-ml/src/main/scala/org/apache/flink/ml/nn/QuadTree.scala
index 95a1771..bbccd71 100644
--- a/flink-libraries/flink-ml/src/main/scala/org/apache/flink/ml/nn/QuadTree.scala
+++ b/flink-libraries/flink-ml/src/main/scala/org/apache/flink/ml/nn/QuadTree.scala
@@ -102,8 +102,8 @@ class QuadTree(
       }.sum
 
       distMetric match {
-        case _: SquaredEuclideanDistanceMetric => minDist
         case _: EuclideanDistanceMetric => math.sqrt(minDist)
+        case _: SquaredEuclideanDistanceMetric => minDist
         case _ => throw new IllegalArgumentException(s" Error: metric must be" +
           s" Euclidean or SquaredEuclidean!")
       }