[GitHub] [spark] wangyum commented on pull request #40555: [SPARK-42926][BUILD][SQL] Upgrade Parquet to 1.13.0

2023-04-14 Thread via GitHub
wangyum commented on PR #40555: URL: https://github.com/apache/spark/pull/40555#issuecomment-1509444887 Merged to master. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

[GitHub] [spark] wangyum commented on pull request #40555: [SPARK-42926][BUILD][SQL] Upgrade Parquet to 1.13.0

2023-04-10 Thread via GitHub
wangyum commented on PR #40555: URL: https://github.com/apache/spark/pull/40555#issuecomment-1502458598 > BTW, if you mind, please revise the PR description. > > 1. Removing `Maybe it can improve read performance.` from the PR description. > 2. Coping [[SPARK-42926][BUILD][SQL] Upg

[GitHub] [spark] wangyum commented on pull request #40555: [SPARK-42926][BUILD][SQL] Upgrade Parquet to 1.13.0

2023-04-10 Thread via GitHub
wangyum commented on PR #40555: URL: https://github.com/apache/spark/pull/40555#issuecomment-1502427643 @dongjoon-hyun Yes. It's no noticeable significant perf difference. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

[GitHub] [spark] wangyum commented on pull request #40555: [SPARK-42926][BUILD][SQL] Upgrade Parquet to 1.13.0

2023-04-10 Thread via GitHub
wangyum commented on PR #40555: URL: https://github.com/apache/spark/pull/40555#issuecomment-1501603107 TPC-DS benchmark result: Query | Parquet 1.13.0(first time) | Parquet 1.12.3(first time) | Parquet 1.13.0(second time) | Parquet 1.12.3(second time) | Parquet 1.13.0(third time) | Parq