Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/6796#issuecomment-118730215
Hey @rtreffer, just want to make sure whether you are still working on
this? I'm asking because I just opened #7231 to refactor Parquet read path for
interoperability and backwards-compatibility, which also touches the decimal
parts. I believe the new [`CatalystDecimalConverter`] [1] already covers the
read path of decimals with precision > 18, which means this PR can be further
simplified. Just in case you don't have time to continue this PR, I'm happy to
fork your branch and get it merged (will still list you as the main author).
[1]:
https://github.com/apache/spark/pull/7231/files#diff-1d6c363c04155a9328fe1f5bd08a2f90R237
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
