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

Add the proxy-mode stateless test #3800

Merged
merged 6 commits into from
Jan 8, 2022

Conversation

BohuTANG
Copy link
Member

@BohuTANG BohuTANG commented Jan 7, 2022

I hereby agree to the terms of the CLA available at: https://databend.rs/dev/policies/cla/

Summary

Part of #3740

  1. change USE TENANT to SUDO USE TENANT since mysql driver will handle the USE statement to USE 'TENANT xx' as use database
  2. Add proxy-mode stateless test
  3. Refine the test suites directory structure

Changelog

  • Improvement

Related Issues

Part of #3740

Test Plan

Unit Tests

Stateless Tests

@databend-bot
Copy link
Member

Thanks for the contribution!
I have applied any labels matching special text in your PR Changelog.

Please review the labels and make any necessary changes.

@vercel
Copy link

vercel bot commented Jan 7, 2022

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/databend/databend/FGVh1CfyLyBDCF3uGzG8AjeExoBj
✅ Preview: Canceled

[Deployment for f84447e canceled]

@BohuTANG BohuTANG marked this pull request as ready for review January 7, 2022 12:18
@sundy-li
Copy link
Member

sundy-li commented Jan 7, 2022

sudo use tenent is a little hacky.

How about improving it by set tenent 'xxx' .

@BohuTANG
Copy link
Member Author

BohuTANG commented Jan 7, 2022

sudo use tenent is a little hacky.

How about improving it by set tenent 'xxx' .

sudo use tenant is a administration command and only used in proxy mode(not for user), it mainly for a client to send commands to metasrv, not a user command.
Though, we should sperate it from the user command.

@BohuTANG
Copy link
Member Author

BohuTANG commented Jan 8, 2022

This only for proxy-mode and re-struct some stateless, let's merge

@BohuTANG BohuTANG merged commit 3eef033 into databendlabs:main Jan 8, 2022
@BohuTANG BohuTANG deleted the dev-proxy-mode-test branch January 8, 2022 00:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants