[SPARK-33086][FOLLOW-UP] Remove unused Optional import from pyspark.resource.profile stub #30002
Conversation
|
Test build #129629 has finished for PR 30002 at commit
|
|
Kubernetes integration test starting |
|
Kubernetes integration test status success |
|
cc @HyukjinKwon (sorry, I should've seen that). |
HyukjinKwon
left a comment
HyukjinKwon
left a comment
There was a problem hiding this comment.
It's okay since the PR is already open but let's avoid fixing only a couple of in a separate PR in general. They don't affect the main codes and can be fixed together when you touch these codes later.
|
Merged to master. |
…esource.profile stub ### What changes were proposed in this pull request? Remove unused `typing.Optional` import from `pyspark.resource.profile` stub. ### Why are the changes needed? Since SPARK-32319 we don't allow unused imports. However, this one slipped both local and CI tests for some reason. ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? Existing tests and mypy check. Closes apache#30002 from zero323/SPARK-33086-FOLLOWUP. Authored-by: zero323 <mszymkiewicz@gmail.com> Signed-off-by: HyukjinKwon <gurwls223@apache.org>
Noted. I couldn't identify why it didn't trigger error in CI and thought it is better to fix it now, before it starts failing build at some point in the future. Thanks. |
4 participants
What changes were proposed in this pull request?
Remove unused
typing.Optionalimport frompyspark.resource.profilestub.Why are the changes needed?
Since SPARK-32319 we don't allow unused imports. However, this one slipped both local and CI tests for some reason.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Existing tests and mypy check.