-
Notifications
You must be signed in to change notification settings - Fork 371
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
Allow uppercase Latin letters in database names #2451
Comments
Hi @AlekSi, can i take it? |
@syasyayas sure! |
Hi @AlekSi, could you clear what does "For example, it should be possible to create database ABC with PostgreSQL backend, but then it will be the same as abc, aBc, etc" mean as PostgreSQL supports both case sensitive and not case sensitive names like:
|
That's right, and we already should quote names everywhere. That's great! |
Hey @AlekSi! So i try to setup the working environment, but I face the issue with tigris on task env-up command:
Any idea how to fix this? |
@syasyayas Please try |
@AlekSi Already did several times |
Please merge with the current |
this is everything except for docker logs as the didn't fit in my terminal everything except for docker logs
|
And this is docker logs: docker logs
|
Now the full output of |
task env-up logs
|
Oh I think i figured out what was the problem. When I try to start standalone mongodb container this shows up: mongodb warning
I will try to switch the system I am on. |
|
What should be done?
We should allow uppercase Latin letters in a way that the backend supports it. For example, it should be possible to create database
ABC
with PostgreSQL backend,but then it will be the same as.abc
,aBc
, etcWhere?
FerretDB/internal/handlers/pg/pgdb/databases.go
Lines 30 to 31 in cc34dfc
FerretDB/internal/util/testutil/db.go
Lines 66 to 67 in cc34dfc
FerretDB/website/docs/diff.md
Line 29 in cc34dfc
Definition of Done
pgdb
tests updated;The text was updated successfully, but these errors were encountered: