Skip to content

Web-console: remove specific column filters#8343

Merged
fjy merged 5 commits intoapache:masterfrom
implydata:remove-filter-rebased
Aug 20, 2019
Merged

Web-console: remove specific column filters#8343
fjy merged 5 commits intoapache:masterfrom
implydata:remove-filter-rebased

Conversation

@mcbrewster
Copy link
Copy Markdown
Contributor

@mcbrewster mcbrewster commented Aug 20, 2019

Screen Shot 2019-08-19 at 3 33 05 PM

The column tree menu is now aware of the current filters applied in the whereClause of the queryAst. If a column is being used to filter will then be presented with the option to remove that filter

@vogievetsky
Copy link
Copy Markdown
Contributor

LGTM! 👍

@fjy fjy merged commit e64070b into apache:master Aug 20, 2019
@fjy fjy deleted the remove-filter-rebased branch August 20, 2019 02:54
gianm pushed a commit to implydata/druid-public that referenced this pull request Aug 20, 2019
* add clear filter

* update tool kit

* remove usless check

* auto run

* add %
gianm added a commit that referenced this pull request Aug 21, 2019
* Docusaurus build framework + ingestion doc refresh.

* stick to npm instead of yarn

* fix typos

* restore some _bin

* Adjustments.

* detect and fix redirect anchors

* update anchor lint

* Web-console: remove specific column filters (#8343)

* add clear filter

* update tool kit

* remove usless check

* auto run

* add %

* Fix resource leak (#8337)

* Fix resource leak

* Patch comments

* Enable Spotbugs NP_NONNULL_RETURN_VIOLATION (#8234)

* Fixes from PR review.

* Fix more anchors.

* Preamble nix.

* Fix more anchors, headers

* clean up placeholder page

* add to website lint to travis config

* better broken link checking

* travis fix

* Fixed more broken links

* better redirects

* unfancy catch

* fix LGTM error

* link fixes

* fix md issues

* Addl fixes
@clintropolis clintropolis added this to the 0.16.0 milestone Aug 23, 2019
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.

4 participants