chore(preprocessing): Move Transforms and Validators to preprocessing…

… package. (#216)
spinnaker · Jan 24, 2018 · 0509d58 · 0509d58
1 parent 10cdaab
commit 0509d58
Show file tree

Hide file tree

Showing 6 changed files with 11 additions and 4 deletions.
diff --git a/kayenta-judge/src/main/scala/com/netflix/kayenta/judge/NetflixACAJudge.scala b/kayenta-judge/src/main/scala/com/netflix/kayenta/judge/NetflixACAJudge.scala
@@ -23,8 +23,10 @@ import com.netflix.kayenta.canary.{CanaryClassifierThresholdsConfig, CanaryConfi
 import com.netflix.kayenta.judge.classifiers.metric._
 import com.netflix.kayenta.judge.classifiers.score.{ScoreClassification, ThresholdScoreClassifier}
 import com.netflix.kayenta.judge.detectors.IQRDetector
+import com.netflix.kayenta.judge.preprocessing.{Transforms, ValidationResult, Validators}
 import com.netflix.kayenta.judge.scorers.{ScoreResult, WeightedSumScorer}
 import com.netflix.kayenta.judge.stats.DescriptiveStatistics
+import com.netflix.kayenta.judge.utils.MapUtils
 import com.netflix.kayenta.metrics.MetricSetPair
 import com.netflix.kayenta.r.{MannWhitney, RExecutionException}
 import com.typesafe.scalalogging.StrictLogging

diff --git a/...om/netflix/kayenta/judge/Transforms.scala → ...enta/judge/preprocessing/Transforms.scala b/...om/netflix/kayenta/judge/Transforms.scala → ...enta/judge/preprocessing/Transforms.scala
@@ -14,8 +14,9 @@
  * limitations under the License.
  */
 
-package com.netflix.kayenta.judge
+package com.netflix.kayenta.judge.preprocessing
 
+import com.netflix.kayenta.judge.Metric
 import com.netflix.kayenta.judge.detectors.BaseOutlierDetector
 
 

diff --git a/...om/netflix/kayenta/judge/Validators.scala → ...enta/judge/preprocessing/Validators.scala b/...om/netflix/kayenta/judge/Validators.scala → ...enta/judge/preprocessing/Validators.scala
@@ -14,7 +14,9 @@
  * limitations under the License.
  */
 
-package com.netflix.kayenta.judge
+package com.netflix.kayenta.judge.preprocessing
+
+import com.netflix.kayenta.judge.Metric
 
 case class ValidationResult(valid: Boolean, reason: Option[String]=None)
 

diff --git a/.../com/netflix/kayenta/judge/MapUtils.scala → ...etflix/kayenta/judge/utils/MapUtils.scala b/.../com/netflix/kayenta/judge/MapUtils.scala → ...etflix/kayenta/judge/utils/MapUtils.scala
@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-package com.netflix.kayenta.judge
+package com.netflix.kayenta.judge.utils
 
 import scala.collection.JavaConverters._
 

diff --git a/kayenta-judge/src/test/scala/com/netflix/kayenta/judge/MapUtilsSuite.scala b/kayenta-judge/src/test/scala/com/netflix/kayenta/judge/MapUtilsSuite.scala
@@ -18,6 +18,7 @@ package com.netflix.kayenta.judge
 
 import java.util.Collections
 
+import com.netflix.kayenta.judge.utils.MapUtils
 import org.scalatest.FunSuite
 
 class MapUtilsSuite extends FunSuite {

diff --git a/kayenta-judge/src/test/scala/com/netflix/kayenta/judge/TransformSuite.scala b/kayenta-judge/src/test/scala/com/netflix/kayenta/judge/TransformSuite.scala
@@ -16,8 +16,9 @@
 
 package com.netflix.kayenta.judge
 
-import com.netflix.kayenta.judge.Transforms.{removeNaNs, removeOutliers}
+import com.netflix.kayenta.judge.preprocessing.Transforms.{removeNaNs, removeOutliers}
 import com.netflix.kayenta.judge.detectors.{IQRDetector, KSigmaDetector}
+import com.netflix.kayenta.judge.preprocessing.Transforms
 import org.scalatest.FunSuite