Skip to content

feat: dataset REST API for distinct values#10595

Merged
dpgaspar merged 7 commits into
apache:masterfrom
preset-io:feat/api-distinct-field
Aug 17, 2020
Merged

feat: dataset REST API for distinct values#10595
dpgaspar merged 7 commits into
apache:masterfrom
preset-io:feat/api-distinct-field

Conversation

@dpgaspar
Copy link
Copy Markdown
Member

@dpgaspar dpgaspar commented Aug 13, 2020

SUMMARY

Creates a new generic endpoint to query distinct values from a ModelRestApi useful for filter dropdowns

This endpoint behaves in a very similar way then /api/v1/<resource>/related/<column_name> on this case /api/v1/<resource>/distinct/<column_name>

Accepts also the same JSON/Rison query parameters:

{
  "filter": "string",
  "page": 0,
  "page_size": 0
}

To enable this generic endpoint on a ModelRestApi child class we need to:

include_route_methods = { ..., RouteMethod.DISTINCT, }

Enable the fields that we allow to be queried:

allowed_distinct_fields = {"schema"}

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Screenshot 2020-08-13 at 16 48 27

TEST PLAN

ADDITIONAL INFORMATION

  • Has associated issue:
  • Changes UI
  • Requires DB Migration.
  • Confirm DB Migration upgrade and downgrade tested.
  • Introduces new feature or API
  • Removes existing feature or API

@dpgaspar dpgaspar marked this pull request as ready for review August 13, 2020 19:45
@dpgaspar dpgaspar requested review from nytai and villebro August 13, 2020 19:46
@superset-github-bot superset-github-bot Bot added api and removed api labels Aug 13, 2020
Copy link
Copy Markdown
Member

@nytai nytai left a comment

Choose a reason for hiding this comment

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

lgtm!

@dpgaspar dpgaspar merged commit 692266f into apache:master Aug 17, 2020
@dpgaspar dpgaspar deleted the feat/api-distinct-field branch August 17, 2020 14:47
amitmiran137 pushed a commit to amitmiran137/incubator-superset that referenced this pull request Aug 21, 2020
* master: (43 commits)
  feat: Getting fancier with Storybook (apache#10647)
  fix: dedup groupby in viz.py while preserving order (apache#10633)
  feat: bump superset-ui for certified tag (apache#10650)
  feat: setup react page with submenu for datasources listview  (apache#10642)
  feat: add certification to metrics (apache#10630)
  feat(viz-plugins): add date formatting to pivot-table (apache#10637)
  fix: controls scroll issue (apache#10644)
  feat: Allow tests files in  /src (plus Label component tests) (apache#10634)
  fix: remove duplicated params and cache_timeout from list_columns; add viz_type to list_columns (apache#10643)
  chore: splitting button stories into separate stories (apache#10631)
  refactor: remove slice level label_colors from dashboard init load (apache#10603)
  feat: card view bulk select (apache#10607)
  style: Label styling/storybook touchups (apache#10627)
  fix: removing unsupported modal sizes (apache#10625)
  feat(datasource): remove deleted columns and update column type on metadata refresh (apache#10619)
  improve documentation for country maps (apache#10621)
  chore: npm audit fix as of 2020-08-15 (apache#10613)
  feat: dataset REST API for distinct values (apache#10595)
  chore: bump react-redux to 5.1.2, whittling console noise (apache#10602)
  fixing console error about bad html attribute (apache#10604)
  ...

# Conflicts:
#	superset-frontend/src/explore/components/ExploreViewContainer.jsx
#	superset-frontend/src/views/App.tsx
#	superset/config.py
Ofeknielsen pushed a commit to ofekisr/incubator-superset that referenced this pull request Oct 5, 2020
* feat: dataset REST API for distinct values

* add tests and fix lint

* fix mypy, and tests

* fix docs

* fix test

* lint

* fix test
auxten pushed a commit to auxten/incubator-superset that referenced this pull request Nov 20, 2020
* feat: dataset REST API for distinct values

* add tests and fix lint

* fix mypy, and tests

* fix docs

* fix test

* lint

* fix test
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.38.0 First shipped in 0.38.0 labels Mar 12, 2024
qfcwell pushed a commit to qfcwell/superset that referenced this pull request May 12, 2026
* feat: dataset REST API for distinct values

* add tests and fix lint

* fix mypy, and tests

* fix docs

* fix test

* lint

* fix test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels size/L 🚢 0.38.0 First shipped in 0.38.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants