-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Improved display of multidimensional partitions #16999
Comments
@sryza Thoughts on these suggestions? |
I wouldn't rule this out in the long term, but this change would have big implications so would like to punt on it for now.
This seems like a good idea to me.
This also seems like a good idea to me. |
I'll take a look at adding some search inputs on there. It sounds like some of @salazarm's recent work may resolve the truncation and line wrapping. |
Just to note, many of these usability issues also apply in the case of singly-partitioned assets. We have several thousand dynamic partitions, with moderately long keys that tend to differ at the end of the string. It's quite difficult to find a single partition. Filtering/searching partition keys would be ideal, and the ability to increase the width of the partition key column to see the full key would also be great. Thanks! |
@FlightOfStairs @ztessler Are you still seeing the wrapping issue? We have middle truncation in place as of #16961, so ideally that should be working correctly now. |
@hellendag yes, seeing middle truncation now on 1.5.6, much more usable, thanks! |
What's the use case?
The current catalog is awkward to use for multi-partitioned assets with high cardinality or long names. We have partitions by company and date.
Our company partition is made up of
<slug>:<uuid>
, e.g.,mutiny-hq:c528a915-3339-4734-886d-ac3a03f65e22
. The slug is a human-readable name, while uuid is required for the op. These are dynamic partitions, so we can’t easily create a static mapping from slug to ID, as new companies will be automatically added.Ideas of implementation
Any of these would greatly help us:
white-space: nowrap;
.Additional information
No response
Message from the maintainers
Impacted by this issue? Give it a 👍! We factor engagement into prioritization.
The text was updated successfully, but these errors were encountered: