Skip to content

Conversation

@Pazus
Copy link

@Pazus Pazus commented Aug 25, 2019

Fix proposal based on #156

@pivotal-issuemaster
Copy link

@Pazus Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@pivotal-issuemaster
Copy link

@Pazus Thank you for signing the Contributor License Agreement!

@schauder
Copy link
Contributor

I agree by now with @mp911de opinion that

Allowing escape chars to be part of the column name opens a can of worms and can lead to ambiguity amongst column aliases as we remove characters relying on some sort of uniqueness amongst column names.

And rather not following the road of trying to remove special characters from column names.

@schauder schauder closed this Aug 26, 2019
mp911de pushed a commit that referenced this pull request Feb 21, 2022
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

Successfully merging this pull request may close these issues.

3 participants