spark icon indicating copy to clipboard operation
spark copied to clipboard

[SPARK-39892][SQL] Use ArrowType.Decimal(precision, scale, bitWidth) instead of ArrowType.Decimal(precision, scale)

Open panbingkun opened this issue 2 years ago • 1 comments

What changes were proposed in this pull request?

Use ArrowType.Decimal(precision, scale, bitWidth) instead of ArrowType.Decimal(precision, scale) to cleanup following compilation warnings:

[warn] /home/runner/work/spark/spark/sql/catalyst/src/main/scala/org/apache/spark/sql/util/ArrowUtils.scala:48:49: [deprecation @ org.apache.spark.sql.util.ArrowUtils.toArrowType | origin=org.apache.arrow.vector.types.pojo.ArrowType.Decimal. | version=] constructor Decimal in class Decimal is deprecated

image

Why are the changes needed?

Cleanup deprecation api usage related to Arrow.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Pass GA.

panbingkun avatar Jul 27 '22 07:07 panbingkun

Can one of the admins verify this patch?

AmplabJenkins avatar Jul 27 '22 09:07 AmplabJenkins

We're closing this PR because it hasn't been updated in a while. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable. If you'd like to revive this PR, please reopen it and ask a committer to remove the Stale tag!

github-actions[bot] avatar Nov 05 '22 00:11 github-actions[bot]