You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by "zhenlineo (via GitHub)" <gi...@apache.org> on 2023/07/29 04:03:25 UTC

[GitHub] [spark] zhenlineo opened a new pull request, #42222: [SPARK-43744][CONNECT][Followup]Throw error from the constructor

zhenlineo opened a new pull request, #42222:
URL: https://github.com/apache/spark/pull/42222

   ### What changes were proposed in this pull request?
   Made the stub constructor to throw ClassNotFoundException if called.
   A tiny improvement to not recreate class loaders in executor if stubbing is not enabled.
   
   ### Why are the changes needed?
   Enhancement to https://github.com/apache/spark/pull/42069
   Should be merged to 3.5.
   
   ### Does this PR introduce _any_ user-facing change?
   No
   
   ### How was this patch tested?
   Unit tests.


-- 
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-unsubscribe@spark.apache.org

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


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


[GitHub] [spark] HyukjinKwon commented on pull request #42222: [SPARK-43744][CONNECT][FOLLOW-UP]Throw error from the constructor

Posted by "HyukjinKwon (via GitHub)" <gi...@apache.org>.
HyukjinKwon commented on PR #42222:
URL: https://github.com/apache/spark/pull/42222#issuecomment-1656995285

   Merged to master and branch-3.5.


-- 
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-unsubscribe@spark.apache.org

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


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


[GitHub] [spark] HyukjinKwon closed pull request #42222: [SPARK-43744][CONNECT][FOLLOW-UP]Throw error from the constructor

Posted by "HyukjinKwon (via GitHub)" <gi...@apache.org>.
HyukjinKwon closed pull request #42222: [SPARK-43744][CONNECT][FOLLOW-UP]Throw error from the constructor
URL: https://github.com/apache/spark/pull/42222


-- 
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-unsubscribe@spark.apache.org

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


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