Skip to content
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

gkdbutils for upper-lower cassandra names #189

Open
RoyWilliams opened this issue Sep 2, 2024 · 1 comment
Open

gkdbutils for upper-lower cassandra names #189

RoyWilliams opened this issue Sep 2, 2024 · 1 comment
Assignees

Comments

@RoyWilliams
Copy link
Contributor

Now that we have changed to using upper-lower case attribute names in cassandra, we should not be making them lower-case, since the database will no longer recognise the name. The ingest breaks because of Line 187 of ingestGenericDatabaseTable in gkdbutils, where it lower-cases everything:
lckeys = ",".join([k.lower().replace('-','').replace('/','') for k in keys])

In the new way of doing things, this should be instead just doublequoting the names:
lckeys = ",".join(['"' + k + '"' for k in keys])

@genghisken
Copy link
Collaborator

Code is pushed. Note that instead of the above, I've done the following, which excludes hyphens or slashes in the keys.

lckeys = ",".join(['"' + k.replace('-','').replace('/','') + '"' for k in keys])

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants