Skip to content
New issue

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

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

Already on GitHub? # to your account

[BUG] CI build failure with 341db: method getKillReason has weaker access privileges; it should be public #10063

Closed
andygrove opened this issue Dec 15, 2023 · 0 comments · Fixed by #10070
Assignees
Labels
bug Something isn't working build Related to CI / CD or cleanly building

Comments

@andygrove
Copy link
Contributor

andygrove commented Dec 15, 2023

Describe the bug
I just ran into this error in CI building against 341db.

[ERROR] /home/ubuntu/spark-rapids/tests/src/test/scala/org/apache/spark/sql/rapids/metrics/source/MockTaskContext.scala:69: 

method getKillReason has weaker access privileges; it should be public

overriding method getKillReason in class TaskContext of type ()Option[String];

Steps/Code to reproduce bug
Ci build with [datrabricks]

Expected behavior

Environment details (please complete the following information)

Additional context

@andygrove andygrove added bug Something isn't working ? - Needs Triage Need team to review and classify labels Dec 15, 2023
@andygrove andygrove self-assigned this Dec 15, 2023
@andygrove andygrove changed the title [BUG] CI build faliure with 341db: method getKillReason has weaker access privileges; it should be public [BUG] CI build failure with 341db: method getKillReason has weaker access privileges; it should be public Dec 15, 2023
@andygrove andygrove added the build Related to CI / CD or cleanly building label Dec 15, 2023
@sameerz sameerz removed the ? - Needs Triage Need team to review and classify label Dec 19, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working build Related to CI / CD or cleanly building
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants