You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2021/03/09 10:34:34 UTC

[GitHub] [spark] maropu commented on pull request #31785: [SPARK-33498][SQL][TESTS][FOLLOWUP] Remove SQLConf.withExistingConf in CastSuite

maropu commented on pull request #31785:
URL: https://github.com/apache/spark/pull/31785#issuecomment-793688261


   > There is a conflict on branch-3.1. Could you make a backporting PR?
   
   Thanks for your quick review&merge! I'll open a PR for that later.
   
   > I realized that this is a follow-up of old SPARK-33498 who fixed version is 3.1.0. It looks a little weird because this goes only to 3.2.0 and 3.1.2.
   
   Ah, yea, as you said, it looks a bit weird. But, this is a minor update, so IMO it is okay as it is. Or, do you think we need to do anything for that?


----------------------------------------------------------------
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.

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