holdenk commented on PR #54523: URL: https://github.com/apache/spark/pull/54523#issuecomment-3994232202
That is better but I don't love a hard fail with a cryptic message even if it says here's to turn it off it's saying you can turn it off but YOLO. If you think we really need a hard fail with work around allowlist how about a more specific message saying _why_ and that there's a solution coming (and existing code should still be fine)? To be clear I don't like this solution but I won't block it if others want to merge it (I would block hard fail no way to disable no alternate API). -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
