zero323 commented on pull request #34671: URL: https://github.com/apache/spark/pull/34671#issuecomment-975825755
> The number of `ignore[attr-defined]` hints required seems a little wrong. But I suppose addressing that would require changes to `SparkContext`, which is out of scope for this PR. That's not optimal, but expected (see https://github.com/apache/spark/pull/34680#issuecomment-975397090). If you encounter case where there is no ongoing migration work and you can avoid ignores, it should be OK to extend the stub. Otherwise, we'll do another pass (ignores on `_jvm` are likely to stay, even if different error code, because we cannot lurk into JVM at this stage). -- 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]
