-
Notifications
You must be signed in to change notification settings - Fork 39
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
Citext support on Postgres #40
Comments
ok, thx, i'll look into it. |
i'm not quite sure i want to add support for a database specific datatype, that needs to be explicity enabled (postgres module). You can easily get the same (search-wise) via a functional index. Why not use that instead? |
Understandable, I think I went for
|
ok, well, then we probably need to define datatype aliases/mappings or some kind of plugin architecture, such that the main code base stays clean. However, the benefit of adding support for aliases/mappings seems limited. Take #28 for instance, which can't be easily mapped to an existing datatype due to the |
Citext column type is useful on Postgres for case-insesitive comparisons [0], currently searching for such column fails with
placing following code in initializer fixes it (I could add a PR but not sure about tests)
[0] https://www.postgresql.org/docs/10/static/citext.html
The text was updated successfully, but these errors were encountered: