[jira] [Commented] (SPARK-30170) Eliminate warnings: part 1

["Maxim Gekk (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/SPARK-30170?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16990989#comment-16990989 ] 

Maxim Gekk commented on SPARK-30170:
------------------------------------

I am working on this

> Eliminate warnings: part 1
> --------------------------
>
>                 Key: SPARK-30170
>                 URL: https://issues.apache.org/jira/browse/SPARK-30170
>             Project: Spark
>          Issue Type: Sub-task
>          Components: SQL
>    Affects Versions: 3.0.0
>            Reporter: Maxim Gekk
>            Priority: Minor
>
> Eliminate compilation warnings in:
> # StopWordsRemoverSuite
> {code}
> 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]) =>
> {code}
> # MLTest.scala
> {code}
>     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 {
> {code}
> # FloatType.scala
> {code}
> 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
> {code}
> # AnalysisExternalCatalogSuite.scala
> {code}
> Warning:Warning:line (62)method verifyZeroInteractions in class Mockito is deprecated: see corresponding Javadoc for more information.
>       verifyZeroInteractions(catalog)
> {code}
> # CSVExprUtilsSuite.scala
> {code}
> Warning:Warning:line (81)Octal escape literals are deprecated, use \u0000 instead.
>     ("\0", Some("\u0000"), None)
> {code}
> # CollectionExpressionsSuite.scala, ashExpressionsSuite.scala, ExpressionParserSuite.scala 
> {code}
> 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)
> {code}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@spark.apache.org
For additional commands, e-mail: issues-help@spark.apache.org