Skip to content

table-filter: implement the new table filter#341

Merged
kennytm merged 3 commits intomasterfrom
kennytm/table-filter
May 15, 2020
Merged

table-filter: implement the new table filter#341
kennytm merged 3 commits intomasterfrom
kennytm/table-filter

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