Skip to content

Conversation

@TCeason
Copy link
Collaborator

@TCeason TCeason commented Mar 22, 2023

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

Summary

add more test for grant privileges

Closes #issue

@vercel
Copy link

vercel bot commented Mar 22, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated
databend ⬜️ Ignored (Inspect) Mar 22, 2023 at 8:00AM (UTC)

@mergify
Copy link
Contributor

mergify bot commented Mar 22, 2023

This pull request's title is not fulfill the requirements. @TCeason please update it 🙏.

Valid format:

fix(query): fix group by string bug
  ^         ^---------------------^
  |         |
  |         +-> Summary in present tense.
  |
  +-------> Type: rfc, feat, fix, refactor, ci, docs, chore

Valid types:

  • rfc: this PR proposes a new RFC
  • feat: this PR introduces a new feature to the codebase
  • fix: this PR patches a bug in codebase
  • refactor: this PR changes the code base without new features or bugfix
  • ci: this PR changes build/testing/ci steps
  • docs: this PR changes the documents or websites
  • chore: this PR only has small changes that no need to record

@TCeason TCeason changed the title test(query): add more test for grant privileges ci(query): add more test for grant privileges Mar 22, 2023
@mergify mergify bot added the pr-build this PR changes build/testing/ci steps label Mar 22, 2023
@TCeason TCeason force-pushed the add_more_test_for_priv branch 4 times, most recently from f20b9bd to 6305279 Compare March 22, 2023 07:22
@TCeason TCeason force-pushed the add_more_test_for_priv branch from 6305279 to d13e72a Compare March 22, 2023 07:59
@BohuTANG
Copy link
Member

There is a comment about the privilege which caused CI failure many times:
#10708 (comment)

@BohuTANG BohuTANG merged commit 844ecdf into databendlabs:main Mar 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-build this PR changes build/testing/ci steps

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants