Skip to content

Conversation

@Fokko
Copy link
Contributor

@Fokko Fokko commented Sep 1, 2020

What changes were proposed in this pull request?

Add a missing import for Python.

Why are the changes needed?

Otherwise, it will crash :)

Does this PR introduce any user-facing change?

No.

How was this patch tested?

In master there is a Flake8 check, this is just a backport.

@SparkQA
Copy link

SparkQA commented Sep 1, 2020

Test build #128159 has finished for PR 29615 at commit e9edb6f.

  • This patch fails MiMa tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@HyukjinKwon
Copy link
Member

Let me close this one, @Fokko. Sorry for a false alarm.

@HyukjinKwon HyukjinKwon closed this Sep 2, 2020
@Fokko
Copy link
Contributor Author

Fokko commented Sep 2, 2020

Ah no worries, maybe a good thing to not use wildcards anymore :)

@Fokko Fokko deleted the fd-add-missing-imports-branch-3-0 branch September 2, 2020 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants