[SPARK-30170] Eliminate warnings: part 1

• Type: Sub-task
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.0.0
• Fix Version/s: 3.0.0
• Component/s: SQL
• Assignee: Max Gekk
• Reporter: Max Gekk
• Created: 08/Dec/19 20:49
• Updated: 12/Dec/19 14:41
• Resolved: 12/Dec/19 14:41

Description

Eliminate compilation warnings in:

1. StopWordsRemoverSuite

   Warning:Warning:line (245)non-variable type argument String in type pattern Seq[String] (the underlying of Seq[String]) is unchecked since it is eliminated by erasure
           case Row(r1: Seq[String], e1: Seq[String], r2: Seq[String], e2: Seq[String]) =>
       Warning:Warning:line (245)non-variable type argument String in type pattern Seq[String] (the underlying of Seq[String]) is unchecked since it is eliminated by erasure
           case Row(r1: Seq[String], e1: Seq[String], r2: Seq[String], e2: Seq[String]) =>
       Warning:Warning:line (245)non-variable type argument String in type pattern Seq[String] (the underlying of Seq[String]) is unchecked since it is eliminated by erasure
           case Row(r1: Seq[String], e1: Seq[String], r2: Seq[String], e2: Seq[String]) =>
       Warning:Warning:line (245)non-variable type argument String in type pattern Seq[String] (the underlying of Seq[String]) is unchecked since it is eliminated by erasure
           case Row(r1: Seq[String], e1: Seq[String], r2: Seq[String], e2: Seq[String]) =>
       Warning:Warning:line (271)non-variable type argument String in type pattern Seq[String] (the underlying of Seq[String]) is unchecked since it is eliminated by erasure
           case Row(r1: Seq[String], e1: Seq[String], r2: Seq[String], e2: Seq[String]) =>
       Warning:Warning:line (271)non-variable type argument String in type pattern Seq[String] (the underlying of Seq[String]) is unchecked since it is eliminated by erasure
           case Row(r1: Seq[String], e1: Seq[String], r2: Seq[String], e2: Seq[String]) =>
       Warning:Warning:line (271)non-variable type argument String in type pattern Seq[String] (the underlying of Seq[String]) is unchecked since it is eliminated by erasure
           case Row(r1: Seq[String], e1: Seq[String], r2: Seq[String], e2: Seq[String]) =>
       Warning:Warning:line (271)non-variable type argument String in type pattern Seq[String] (the underlying of Seq[String]) is unchecked since it is eliminated by erasure
           case Row(r1: Seq[String], e1: Seq[String], r2: Seq[String], e2: Seq[String]) =>
   

1. MLTest.scala

       Warning:Warning:line (88)match may not be exhaustive.
   It would fail on the following inputs: NumericAttribute(), UnresolvedAttribute
       val n = Attribute.fromStructField(dataframe.schema(colName)) match {
   

1. FloatType.scala

   Warning:Warning:line (81)method apply in object BigDecimal is deprecated (since 2.11.0): The default conversion from Float may not do what you want. Use BigDecimal.decimal for a String representation, or explicitly convert the Float with .toDouble.
       def quot(x: Float, y: Float): Float = (BigDecimal(x) quot BigDecimal(y)).floatValue
       Warning:Warning:line (81)method apply in object BigDecimal is deprecated (since 2.11.0): The default conversion from Float may not do what you want. Use BigDecimal.decimal for a String representation, or explicitly convert the Float with .toDouble.
       def quot(x: Float, y: Float): Float = (BigDecimal(x) quot BigDecimal(y)).floatValue
       Warning:Warning:line (82)method apply in object BigDecimal is deprecated (since 2.11.0): The default conversion from Float may not do what you want. Use BigDecimal.decimal for a String representation, or explicitly convert the Float with .toDouble.
       def rem(x: Float, y: Float): Float = (BigDecimal(x) remainder BigDecimal(y)).floatValue
       Warning:Warning:line (82)method apply in object BigDecimal is deprecated (since 2.11.0): The default conversion from Float may not do what you want. Use BigDecimal.decimal for a String representation, or explicitly convert the Float with .toDouble.
       def rem(x: Float, y: Float): Float = (BigDecimal(x) remainder BigDecimal(y)).floatValue
   

1. AnalysisExternalCatalogSuite.scala

   Warning:Warning:line (62)method verifyZeroInteractions in class Mockito is deprecated: see corresponding Javadoc for more information.
         verifyZeroInteractions(catalog)
   

1. CSVExprUtilsSuite.scala

   Warning:Warning:line (81)Octal escape literals are deprecated, use \u0000 instead.
       ("\0", Some("\u0000"), None)
   

1. CollectionExpressionsSuite.scala, HashExpressionsSuite.scala, ExpressionParserSuite.scala

   Warning:Warning:line (39)implicit conversion method stringToUTF8Str should be enabled
   by making the implicit value scala.language.implicitConversions visible.
   This can be achieved by adding the import clause 'import scala.language.implicitConversions'
   or by setting the compiler option -language:implicitConversions.
   See the Scaladoc for value scala.language.implicitConversions for a discussion
   why the feature should be explicitly enabled.
     implicit def stringToUTF8Str(str: String): UTF8String = UTF8String.fromString(str)
   

Issue Links

• links to: GitHub Pull Request #26799