Skip to content

Add an api to get all lookup specs - #7025

Merged
fjy merged 2 commits into
apache:masterfrom
jihoonson:get-all-lookups
Feb 8, 2019
Merged

fjy merged 2 commits into
apache:masterfrom
jihoonson:get-all-lookups

Conversation

@jihoonson

Copy link
Copy Markdown
Contributor

Currently it needs 3 APIs to retrieve all lookup specs which is not good for some use cases like UI to show all configured specs.

@fjy

fjy commented Feb 7, 2019

Copy link
Copy Markdown
Contributor

👍

@surekhasaharan surekhasaharan left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM !

@vogievetsky

Copy link
Copy Markdown
Contributor

Should this update the docs here: http://druid.io/docs/latest/querying/lookups.html ?

@jihoonson

Copy link
Copy Markdown
Contributor Author

@vogievetsky thanks, updated the doc.

@fjy fjy added this to the 0.14.0 milestone Feb 8, 2019
@fjy
fjy merged commit d42de57 into apache:master Feb 8, 2019
jihoonson added a commit to jihoonson/druid that referenced this pull request Feb 20, 2019
* Add an api to get all lookup specs

* add doc
fjy pushed a commit that referenced this pull request Feb 20, 2019
* Add an api to get all lookup specs

* add doc
riovic918data pushed a commit to riovic918data/druid that referenced this pull request Jun 12, 2026
* Add an api to get all lookup specs

* add doc
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