Table name issue system is case-insensitive but tableCreate is case-sensitive

[coffeemug]

1. Run r.tableCreate('bar')
2. Run r.tableCreate('Bar')

The query language happily creates both tables, but the cluster immediately raises an issue that there is a name conflict.

After some discussion with @jdoliner and @mlucy and @srh we decided to fix the issue resolution system to be case-sensitive.

[jdoliner]

This is in code review 553. I noticed while testing this that the web UI won't let you create the table "foo" if you already have a table "Foo". We should change that to behavior too just for consistency sake.

[neumino]

Reassigning to myself.
I'll update the web ui when I will have a few minutes.

[jdoliner]

K. This is in essence assigned to both of us since I haven't actually committed my part to next yet but whatever we all know what's going on here.

[jdoliner]

My part of this is done. Reviewed by @Tryneus. @neumino make sure to note when this will ship once you complete your part.

[neumino]

Just to be sure:
Databases AND tables names are now case sensitive right?

[jdoliner]

Yup.
On May 21, 2013 11:21 AM, "Michel" notifications@github.com wrote:

Just to be sure:
Databases AND tables names are now case sensitive right?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/822#issuecomment-18227008
.

[neumino]

Done in 52d5343
Code review 559
Branch michel_822_case_sensitive_name

[neumino]

Review done.
Merged in next with commit 950b622

@jdoliner is your part merged in next too? If yes, we can close this issue I think.

[jdoliner]

Yup it's in next. I just tried testing the webUI and it still wasn't letting me make tables with names that only differed in capitalization. I tried to do shift + f5 to clear the cache but it still didn't work. I'm betting that something weird is happening here, perhaps with the rebuilding of web assets. Can you confirm that this works in your tests?

[neumino]

Err, I'm a moron, I forgot to push on github -_-

I can confirm that it works fine.