fix(GetConstraintsPertab): Also work with non-lowercase column names #10
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Fix for uppercase colum + Uppercase columns are being converted to lowercase and hence error'ng out. this is fixed + Bump the version of the mock-data tool * version bump
|
Done. Still a bit confused how the problem I tried to solve is solved via the change you added, but - if it works it works 🤷♀ |
|
The issue is the columns case sensitivity is kept till it reached the copy statement, during the execution postgres replaces the uppercase with lower case, so added double quotes at copy statement for column to keep the case sensitivity. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Possibly closes #6
(Sorry about the linebreak thing at the end of the file, GH UI doesn't let me not have one when I edit files there.)