gaogaotiantian commented on PR #53412: URL: https://github.com/apache/spark/pull/53412#issuecomment-3640684658
Okay this is a real lint issue - so I believe there is some racing on master, which made the PR in without being checked by ruff. When https://github.com/apache/spark/commit/19a1da938a61bfaffd0ddef35e66a4c41f80d387 is merged, it removed the `pa` usage from that method so `pa` is not needed anymore. Notice that there is still a `pa` in type hint - that does not need `pa` here because `pa` is imported globally for type checking. ```python if TYPE_CHECKING: import pandas as pd import pyarrow as pa ``` This is not a false alarm, this is an issue that should've been caught by ruff if the order is correct. -- 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]
