cloud-fan commented on issue #26847: [SPARK-30214][SQL] A new framework to resolve v2 commands URL: https://github.com/apache/spark/pull/26847#issuecomment-572872920 I like the proposal from @gatorsmile . Let's put it in the [review guideline](https://spark.apache.org/contributing.html) and make it a policy. But the new policy should apply to new PRs only, not already merged PRs. @rdblue do you have some concrete concerns about what's wrong with the new framework? Both @imback82 and I have taken a close look, and we think it's the right direction to go. If the new framework does have some major flaws, then let's revert it. Note that, the DDL/DML command resolution framework is new in 3.0. The v1 commands just look up relation by themselves. If we all agree to use this new framework, we should make sure 3.0 and 3.1 use the same framework, otherwise it's really painful to backport bug fixes in the future.
---------------------------------------------------------------- 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] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
