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

Cast to TEXT applied to UUID columns in the UI filter #184

Closed
vlad-at-yteq opened this Issue Feb 15, 2016 · 1 comment

Comments

Projects
None yet
2 participants
@vlad-at-yteq

vlad-at-yteq commented Feb 15, 2016

Hi Jakob,

First of all congrats, Postico is a great tool!

There is an issue in the UI filter on UUID columns which make querying very slow on sizeable amount of data. t the moment it's done this way:

uuid_col::text = ''

The ::text cast causes UUID index to be skipped. So it should rather do the following:

uuid_col = ''

Postgres will apply implicit cast to '' as ''::uuid.

I would appreciate if this could be fix (I imagine a fix would be fairly trivial).

@jakob jakob added the started label Mar 2, 2016

@jakob

This comment has been minimized.

Show comment
Hide comment
@jakob

jakob Mar 4, 2016

Owner

Hi! Postico 1.0.5 now treats UUIDs as numbers when filtering, which should fix this issue. Thanks for reporting it!

Owner

jakob commented Mar 4, 2016

Hi! Postico 1.0.5 now treats UUIDs as numbers when filtering, which should fix this issue. Thanks for reporting it!

@jakob jakob closed this Mar 4, 2016

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