Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix NPE and incorrect comment #2827

Merged
merged 2 commits into from
Jul 4, 2024
Merged

fix NPE and incorrect comment #2827

merged 2 commits into from
Jul 4, 2024

Conversation

pawel-big-lebowski
Copy link
Collaborator

Recent upgrade of Spark versions introduced NPE which didn't break the tests.

@boring-cyborg boring-cyborg bot added area:integration/spark area:tests Testing code language:java Uses Java programming language labels Jul 3, 2024
Signed-off-by: Pawel Leszczynski <[email protected]>
@pawel-big-lebowski pawel-big-lebowski force-pushed the spark/fix-upgrade-change branch from 623d1a0 to cdf0deb Compare July 3, 2024 18:02
@pawel-big-lebowski pawel-big-lebowski changed the title fix NPE and inccorec comment fix NPE and inccorect comment Jul 4, 2024
@pawel-big-lebowski pawel-big-lebowski marked this pull request as ready for review July 4, 2024 13:03
@mobuchowski mobuchowski changed the title fix NPE and inccorect comment fix NPE and incorrect comment Jul 4, 2024
@pawel-big-lebowski pawel-big-lebowski requested a review from a team as a code owner July 4, 2024 13:03
@mobuchowski mobuchowski merged commit a1648a8 into main Jul 4, 2024
2 of 4 checks passed
@mobuchowski mobuchowski deleted the spark/fix-upgrade-change branch July 4, 2024 13:03
lmassaoy pushed a commit to nubank/NuOpenLineage that referenced this pull request Jul 5, 2024
Signed-off-by: Pawel Leszczynski <[email protected]>
Co-authored-by: Maciej Obuchowski <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:integration/spark area:tests Testing code language:java Uses Java programming language
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants