Skip to content

Conversation

@TCeason
Copy link
Collaborator

@TCeason TCeason commented Jan 8, 2025

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

Summary

create user a identified by '123' with default_role='a';
create role a;
create database a;
grant ownership on a.* to role a;
grant role a to a;
create table a.b(id int);
create role b;
grant ownership on a.b to role b;

-- login with user a
-- this query should return right result
select name, owner from system.tables where database = 'a'and name = 'b'

Tests

  • Unit Test
  • Logic Test
  • Benchmark Test
  • No Test - Explain why

Type of change

  • Bug Fix (non-breaking change which fixes an issue)
  • New Feature (non-breaking change which adds functionality)
  • Breaking Change (fix or feature that could cause existing functionality not to work as expected)
  • Documentation Update
  • Refactoring
  • Performance Improvement
  • Other (please describe):

This change is Reviewable

```sql
create user a identified by '123' with default_role='a';
create role a;
create database a;
grant ownership on a.* to role a;
grant role a to a;
create table a.b(id int);
create role b;
grant ownership on a.b to role b;

-- login with user a
-- this query should return right result
select name, owner from system.tables where database = 'a'and name = 'b'

```
@github-actions github-actions bot added the pr-bugfix this PR patches a bug in codebase label Jan 8, 2025
@TCeason TCeason requested review from b41sh and wubx January 8, 2025 03:26
@TCeason TCeason added the ci-benchmark-local Benchmark: run only local test label Jan 8, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Jan 8, 2025

Docker Image for PR

  • tag: pr-17202-3167ccd-1736309329

note: this image tag is only available for internal use,
please check the internal doc for more details.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 8, 2025

@TCeason TCeason added this pull request to the merge queue Jan 8, 2025
@bohutang bohutang removed this pull request from the merge queue due to a manual request Jan 8, 2025
@bohutang bohutang merged commit 963e27a into databendlabs:main Jan 8, 2025
90 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-benchmark-local Benchmark: run only local test pr-bugfix this PR patches a bug in codebase

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants