-
-
Notifications
You must be signed in to change notification settings - Fork 43
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
citext false positive #89
Comments
Hi @pirj , It makes total sense for me 馃憤 Would you mind to contribute? |
Hi @pirj , I have finally a PR for this (#147) to be fixed but I tested the planner and I think we still would better have a proper (lower) index instead. The reason is that ActiveRecord doesn't recognize
And after adding a lower index:
With that said, I will abandon the PR and close the issue. Please feel free to reopen it. |
Hey @djezzzl 馃憢 Thanks for giving it a shot. |
rails/rails#46568 one done, one to go |
Wow! Well done! Unfortunately, my past experience contributing to ROR wasn't great so I didn't expect it's actually possible 馃榾 |
Wow, you do the magic! Thank you for improving Rails experience for all of us! |
Hey @djezzzl ! 馃憢
I have the following in
db/schema.rb
:Model:
database_consistency
complains:psql mydb
:PostgreSQL
citext
extension docs: https://www.postgresql.org/docs/current/citext.htmlI've debugged it, and:
馃挜
Does it make sense to remove/ignore
lower(
from the index column if this column's type iscitext
?database_consistency
version 1.1.2The text was updated successfully, but these errors were encountered: