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

[AUDIT] uncacheTableOrView changed in CommandUtils #10710

Closed
Tracked by #9259
razajafri opened this issue Apr 15, 2024 · 1 comment · Fixed by #10863
Closed
Tracked by #9259

[AUDIT] uncacheTableOrView changed in CommandUtils #10710

razajafri opened this issue Apr 15, 2024 · 1 comment · Fixed by #10863
Assignees
Labels
audit_4.0.0 Audit related tasks for 4.0.0 bug Something isn't working

Comments

@razajafri
Copy link
Collaborator

Spark made a change in apache/spark@6d41866e631, modifying the parameters to the uncacheTableOrView. We depend on this method and we should make the appropriate change.

@razajafri razajafri added bug Something isn't working ? - Needs Triage Need team to review and classify audit_4.0.0 Audit related tasks for 4.0.0 labels Apr 15, 2024
@razajafri razajafri changed the title uncacheTableOrView changed in CommandUtils [AUDIT] uncacheTableOrView changed in CommandUtils Apr 16, 2024
@mattahrens mattahrens removed the ? - Needs Triage Need team to review and classify label Apr 16, 2024
@mythrocks mythrocks self-assigned this May 22, 2024
mythrocks added a commit to mythrocks/spark-rapids that referenced this issue May 22, 2024
Fixes NVIDIA#10710.

This commit accounts for the changes in the signature of `CommandUtils.uncacheTableOrView`
in Apache Spark 4.0. (See [SPARK-47191](apache/spark#45289).)

Signed-off-by: MithunR <mithunr@nvidia.com>
razajafri pushed a commit that referenced this issue May 30, 2024
…hange. (#10863)

* Account for `CommandUtils.uncacheTableOrView` signature change.

Fixes #10710.

This commit accounts for the changes in the signature of `CommandUtils.uncacheTableOrView`
in Apache Spark 4.0. (See [SPARK-47191](apache/spark#45289).)

Signed-off-by: MithunR <mithunr@nvidia.com>

* Removed unnecessary base class.

---------

Signed-off-by: MithunR <mithunr@nvidia.com>
@mythrocks
Copy link
Collaborator

This has been addressed by #10863.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
audit_4.0.0 Audit related tasks for 4.0.0 bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants