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

Remove redundant BaseMongoConnectorTest #15239

Merged
merged 2 commits into from
Nov 29, 2022
Merged

Conversation

ebyhr
Copy link
Member

@ebyhr ebyhr commented Nov 29, 2022

Description

Additional context and related issues

Release notes

( ) This is not user-visible or docs only and no release notes are required.
( ) Release notes are required, please propose a release note for me.
( ) Release notes are required, with the following suggested text:

# Section
* Fix some things. ({issue}`issuenumber`)

@ebyhr ebyhr added the no-release-notes This pull request does not require release notes entry label Nov 29, 2022
@cla-bot cla-bot bot added the cla-signed label Nov 29, 2022
@ebyhr ebyhr requested a review from Praveen2112 November 29, 2022 13:15
@hashhar
Copy link
Member

hashhar commented Nov 29, 2022

We no longer need it because we test multiple versions via smoke test instead of BCT?

@ebyhr
Copy link
Member Author

ebyhr commented Nov 29, 2022

We no longer need it because we test multiple versions via smoke test instead of BCT?

Correct 🙂

@ebyhr ebyhr merged commit 6f49c4d into trinodb:master Nov 29, 2022
@ebyhr ebyhr deleted the ebi/mongo-test branch November 29, 2022 22:14
@github-actions github-actions bot added this to the 404 milestone Nov 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed no-release-notes This pull request does not require release notes entry
Development

Successfully merging this pull request may close these issues.

2 participants