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

Tables div supports longer names and dedup information #185

Open
nwoolmer opened this issue Aug 14, 2023 · 3 comments
Open

Tables div supports longer names and dedup information #185

nwoolmer opened this issue Aug 14, 2023 · 3 comments
Labels
web-console Issues relevant to "web-console" package

Comments

@nwoolmer
Copy link

nwoolmer commented Aug 14, 2023

Is your feature request related to a problem?

1.

I have tables with long names ("{guid}_{name}"). I have organised them this way round to group similar guids together in the window and the file system. i.e guid_foo and guid_bar will be next to each other for the same guid.

However, using DAY partition and WAL, I can only see the first character of the name, and cannot move the vertical divider to expand the div any further into the code div.

See screenshot for example.

Actioned a while ago, thank you!

It would be useful for me to be able to visualise the upsert keys in some fashion alongside the partitioning/indexing etc.

I appreciate that 7.3.0 only just came out!

Describe the solution you'd like.

For 1.

Freedom to expand the tables div further
Perhaps secondary visuals might work too (i.e tooltip when hovering over etc.)

For 2.

Perhaps icons next to the fields to match indexed symbols and designated timestamps.

Describe alternatives you've considered.

Using shorter names - possible if/when this comes: https://github.com/orgs/questdb/projects/1?pane=issue&itemId=20369114

Reversing the name ordering - would group tables by name instead of guid, which is less ideal for my use case.

Additional context.

image

@insmac insmac transferred this issue from questdb/questdb Aug 14, 2023
@insmac
Copy link
Contributor

insmac commented Aug 14, 2023

Hi @Takheos, many thanks for your feedback! 🙇

Regarding upsert keys, we can add an indication for each column (or an icon) in a row, if it's being used as a deduplication key (similar to how we display designated timestamp indication or column type).

For the long table names problem, we can do a few things, which are not mutually exclusive, they can be added together:

  • Bump the allowed max pane width from 500px to something bigger (we can perhaps make it dynamic to, say, half a viewport width)
  • Add a tooltip with the full table name, visible if the text is longer than its container.

@nwoolmer
Copy link
Author

nwoolmer commented Aug 14, 2023

Hi @insmac , thanks for your time.

Re: upsert keys, I think that could fit nicely with the current style you have for icons and I don't think it'd be too crowded. Not sure what the icon could be. Perhaps a 'D', or an up arrow (upsert key) or something else in your icon pack!

Re: max pane width, I think changing this would be the most immediately useful for me.

Re: tooltip, this could be useful too depending on responsiveness etc. - however, the above change would already be a huge help, and you may find better uses for a tooltip in future (for example, table statistics like number of rows, storage size etc.)

Thanks for your help!

@argshook argshook added the web-console Issues relevant to "web-console" package label Oct 3, 2023
@marregui
Copy link
Contributor

marregui commented Oct 3, 2023

In the meantime, you can query the dedupness of columns with this idiom:

select "column", type, upsertKey from table_columns('signal')

upsert columns will feature a massive True in the 3rd column.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
web-console Issues relevant to "web-console" package
Projects
None yet
Development

No branches or pull requests

4 participants