zero323 edited a comment on pull request #34466:
URL: https://github.com/apache/spark/pull/34466#issuecomment-994232439


   > After this PR, should we open a new Jira ticket to clean up the code by 
removing `# type: ignore [attr -defined]`, which is caused by this PR when not 
merged?
   
   Possibly, but that's not priority at the moment and can be handled after all 
migration tasks are finished (especially when some patterns of handling 
different cases are still emerging). Not to mention that SPARK-37620 touches a 
lot of code and starting another large sweep would cause a lot of issues to be 
resolved (also, it seems like @ByronHsu  already covered a lot of these in 
[that PR](https://github.com/apache/spark/pull/34892/files)).
   
   (For now let's focus on resolving things that are already opened or WIP and 
prioritized, like SPARK-37154, and try to keep migrations in logical order of 
dependencies, so we can reduce further "temp" ignores to the minimum).


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

Reply via email to