databendlabs / databend

๐——๐—ฎ๐˜๐—ฎ, ๐—”๐—ป๐—ฎ๐—น๐˜†๐˜๐—ถ๐—ฐ๐˜€ & ๐—”๐—œ. Modern alternative to Snowflake. Cost-effective and simple for massive-scale analytics. https://databend.com
https://docs.databend.com
Other
7.71k stars 732 forks source link

refactor: remove all share related functions #16383

Closed drmingdrmer closed 3 weeks ago

drmingdrmer commented 3 weeks ago

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

Summary

change: remove all share related functions

The backup of these removed codes is at: https://github.com/datafuselabs/databend/tree/share-bkp Merge this branch to restore the removed codes.

Tests

Type of change

Related Issues


This change isโ€‚Reviewable

drmingdrmer commented 3 weeks ago

@TCeason Please have a look at the statement tests changes, make sure no sql statements are removed except SHARE related sqls. Because some of the statement such as role and share both use grant xxx for privilege management.

Xuanwo commented 3 weeks ago

image

TCeason commented 3 weeks ago

@TCeason Please have a look at the statement tests changes, make sure no sql statements are removed except SHARE related sqls.

Because some of the statement such as role and share both use grant xxx for privilege management.

Ok

TCeason commented 3 weeks ago

@TCeason Please have a look at the statement tests changes, make sure no sql statements are removed except SHARE related sqls.

Because some of the statement such as role and share both use grant xxx for privilege management.

This pr not related to user role privilege.

BohuTANG commented 3 weeks ago

@soyeric128 We can remove the SHARE part from the documention now, thanks.

drmingdrmer commented 3 weeks ago

The backup of these removed codes is at: https://github.com/datafuselabs/databend/tree/share-bkp Merge this branch to restore the removed codes.