Skip to content

Conversation

@imback82
Copy link
Contributor

@imback82 imback82 commented Aug 8, 2021

What changes were proposed in this pull request?

Now that all the commands that use UnresolvedV2Relation have been migrated to use UnresolvedTable and UnresolvedView (e.g, #33200), UnresolvedV2Relation can be removed.

Why are the changes needed?

To remove unused code.

Does this PR introduce any user-facing change?

No

How was this patch tested?

Removing dead code and no code coverage existed before.

@github-actions github-actions bot added the SQL label Aug 8, 2021
@SparkQA
Copy link

SparkQA commented Aug 8, 2021

@SparkQA
Copy link

SparkQA commented Aug 8, 2021

@SparkQA
Copy link

SparkQA commented Aug 8, 2021

Test build #142194 has finished for PR 33677 at commit d63e882.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@imback82
Copy link
Contributor Author

imback82 commented Aug 8, 2021

cc @cloud-fan

@cloud-fan
Copy link
Contributor

thanks, merging to master!

@cloud-fan cloud-fan closed this in b3d7ebb Aug 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants