HeartSaVioR commented on pull request #28523: URL: https://github.com/apache/spark/pull/28523#issuecomment-631145268
Personally I don't see any risk on 1, because we haven't make changes on major release (via merging this in), and master is for next minor release which has more risk to break. Option 2 means we decide to break things again in Spark 3.1, which may only make sense when we make progress on SPARK-31724 in Spark 3.1, but SPARK-31724 isn't necessary needed to depend on this so no point except forcing us to deal with it within Spark 3.1. ---------------------------------------------------------------- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
