Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Schema viewer should expose partitioned columns #185

Closed
fbertsch opened this issue Aug 9, 2017 · 6 comments
Closed

Schema viewer should expose partitioned columns #185

fbertsch opened this issue Aug 9, 2017 · 6 comments
Assignees
Milestone

Comments

@fbertsch
Copy link

fbertsch commented Aug 9, 2017

This only applies to tables which have partitions, i.e. just those in the Hive metastore (unless some RDBMS tables are sharded but... let's not go there). Exposing this information would make it easier to say to people "filter on partitions!" rather than have that be intimate knowledge for each dataset.

@alison985
Copy link

@fbertsch To make sure I'm understanding this right: in Hive data sources display to the user which columns there are partitions on. Perhaps with a [P] designation next to the field name?

@fbertsch
Copy link
Author

fbertsch commented Aug 10, 2017

@alison985 right - here is an example DESCRIBE query output - I've highlighted the partitions descriptions: https://pastebin.mozilla.org/9029373

[P] could work, something that goes along with whatever the schema browser's type information looks like (which I haven't seen yet).

@rafrombrc rafrombrc added this to the 9 milestone Aug 10, 2017
@alison985
Copy link

@fbertsch That pastebin doesn't work for me. Could you try again or another way please?

@fbertsch
Copy link
Author

@alison985 alison985 self-assigned this Aug 14, 2017
alison985 added a commit to alison985/mozilla-redash that referenced this issue Aug 14, 2017
mozilla#185

added to the front of the column name so they are in a consistent
ordinal position for each table

tests pass
@alison985
Copy link

Thanks. PR #201

@madalincm
Copy link

I'm not sure how to verify this bug. @alison985 Can you please provide some steps on how I should proceed verifying this issue?

washort pushed a commit that referenced this issue Dec 13, 2017
#185

added to the front of the column name so they are in a consistent
ordinal position for each table

tests pass
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants