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

table-filter: implement the new table filter #341

Merged
merged 3 commits into from
May 15, 2020
Merged

Conversation

kennytm
Copy link
Contributor

@kennytm kennytm commented May 9, 2020

What problem does this PR solve?

Implement the new style table filter, to be used by BR and Dumpling, and to be replacing black-white-list in Lightning and DM.

What is changed and how it works?

Check List

Tests

  • Unit test

Code changes

Side effects

Related changes

  • Need to update the documentation

overvenus
overvenus previously approved these changes May 13, 2020
Copy link
Member

@overvenus overvenus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@lichunzhu lichunzhu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kennytm kennytm merged commit 330c1a4 into master May 15, 2020
@kennytm kennytm deleted the kennytm/table-filter branch May 15, 2020 12:14
kennytm added a commit that referenced this pull request Jul 8, 2020
* table-filter: implement .gitignore-style table filter

* table-filter: fix build error

Co-authored-by: Chunzhu Li <lichunzhu@stu.xjtu.edu.cn>
kennytm added a commit that referenced this pull request Jul 8, 2020
* table-filter: implement .gitignore-style table filter

* table-filter: fix build error

Co-authored-by: Chunzhu Li <lichunzhu@stu.xjtu.edu.cn>
kennytm added a commit that referenced this pull request Jul 9, 2020
* table-filter: implement the new table filter (#341)

* table-filter: implement .gitignore-style table filter

* table-filter: fix build error

Co-authored-by: Chunzhu Li <lichunzhu@stu.xjtu.edu.cn>

* table-filter: added filter.All() to conveniently make a match-all filter (#347)

* table-filter: added filter.All() to conveniently make a match-all filter

* table-filter: fix: NewSchemasFilter() should return the public interface

* filter,table-filter: replace blacklist/whitelist by blocklist/allowlist (#353)

* filter: fix build error

Co-authored-by: Chunzhu Li <lichunzhu@stu.xjtu.edu.cn>
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