Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/20964#discussion_r178893936
--- Diff: mllib/src/test/scala/org/apache/spark/ml/feature/NGramSuite.scala
---
@@ -84,7 +84,7 @@ class NGramSuite extends MLTest with DefaultReadWriteTest
{
def testNGram(t: NGram, dataFrame: DataFrame): Unit = {
testTransformer[(Seq[String], Seq[String])](dataFrame, t, "nGrams",
"wantedNGrams") {
- case Row(actualNGrams : Seq[String], wantedNGrams: Seq[String]) =>
+ case Row(actualNGrams : Seq[_], wantedNGrams: Seq[_]) =>
--- End diff --
String is not actually checked because of erasure, so IntelliJ complained
with a style warning before this change.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
