Skip to content

GitLab

Explore

Sign in

Primary navigation

Project

M

Metabase
- Activity
- Members
- Labels
- Issues
- Issue boards
- Milestones
- Iterations
- Wiki
- Requirements
- Environments
- Terraform modules
- Incidents
- Service Desk

Snippets Groups Projects

efe86d04

Unverified Commit efe86d04 authored 6 years ago by Cam Saul Committed by GitHub 6 years ago

Downloads
- Plain Diff

Merge pull request #8772 from metabase/remove-duplicate-breakouts-fields-from-fields

Remove duplicate Fields specified in both Fields and Breakouts

parents 3aa1cf35 5de17c88

No related branches found

No related tags found

No related merge requests found

Changes 8

Hide whitespace changes

Inline Side-by-side

Showing

src/metabase/mbql/normalize.clj 32 additions, 1 deletion

src/metabase/mbql/normalize.clj
src/metabase/mbql/schema.clj 15 additions, 6 deletions

src/metabase/mbql/schema.clj
src/metabase/query_processor.clj 14 additions, 7 deletions

src/metabase/query_processor.clj
src/metabase/query_processor/middleware/catch_exceptions.clj 2 additions, 0 deletions

src/metabase/query_processor/middleware/catch_exceptions.clj
test/metabase/api/dataset_test.clj 1 addition, 1 deletion

test/metabase/api/dataset_test.clj
test/metabase/mbql/normalize_test.clj 16 additions, 0 deletions

test/metabase/mbql/normalize_test.clj
test/metabase/query_processor_test/breakout_test.clj 12 additions, 0 deletions

test/metabase/query_processor_test/breakout_test.clj
test/metabase/query_processor_test/failure_test.clj 63 additions, 0 deletions

test/metabase/query_processor_test/failure_test.clj

with 155 additions and 15 deletions

Loading

0% Loading or .

You are about to add 0 people to the discussion. Proceed with caution.

Finish editing this message first!

Please register or sign in to comment