Re: [PR] [SPARK-48169][SPARK-48143][SQL] Revert BadRecordException optimizations [spark]

2024-05-09 Thread via GitHub


cloud-fan closed pull request #46478: [SPARK-48169][SPARK-48143][SQL] Revert 
BadRecordException optimizations
URL: https://github.com/apache/spark/pull/46478


-- 
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 unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



Re: [PR] [SPARK-48169][SPARK-48143][SQL] Revert BadRecordException optimizations [spark]

2024-05-09 Thread via GitHub


cloud-fan commented on PR #46478:
URL: https://github.com/apache/spark/pull/46478#issuecomment-2102119245

   thanks, merging 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 unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



Re: [PR] [SPARK-48169][SPARK-48143][SQL] Revert BadRecordException optimizations [spark]

2024-05-08 Thread via GitHub


dongjoon-hyun commented on PR #46478:
URL: https://github.com/apache/spark/pull/46478#issuecomment-2101213015

   Thank you for adding the link, @vladimirg-db .
   
   cc @cloud-fan and @HyukjinKwon , too


-- 
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 unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



Re: [PR] [SPARK-48169][SPARK-48143][SQL] Revert BadRecordException optimizations [spark]

2024-05-08 Thread via GitHub


vladimirg-db commented on PR #46478:
URL: https://github.com/apache/spark/pull/46478#issuecomment-2101109487

   Sure @dongjoon-hyun! Updated the description


-- 
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 unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[PR] [SPARK-48169][SPARK-48143][SQL] Revert BadRecordException optimizations [spark]

2024-05-08 Thread via GitHub


vladimirg-db opened a new pull request, #46478:
URL: https://github.com/apache/spark/pull/46478

   ### What changes were proposed in this pull request?
   Revert BadRecordException optimizations
   
   
   ### Why are the changes needed?
   To reduce the blast radius - this will be implemented differently
   
   
   ### Does this PR introduce _any_ user-facing change?
   No
   
   
   ### How was this patch tested?
   N/A
   
   
   ### Was this patch authored or co-authored using generative AI tooling?
   No
   


-- 
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 unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org