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

Added escape hatch for IDENTITY() in select into #2055

Conversation

deepakshi-mittal
Copy link
Contributor

@deepakshi-mittal deepakshi-mittal commented Nov 22, 2023

Added escape_hatch_identity_function for IDENTITY() in select into

Need this because identity column will always be the last one in table if created using a select into statement

Task: BABEL-539 and BABEL-4388

Description

Added escape_hatch_identity_function for IDENTITY() in select into
Related pr: #1967

Issues Resolved

BABEL-539, BABEL-4388

Check List

  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.

For more information on following Developer Certificate of Origin and signing off your commits, please check here.

else
{
ereport(ERROR,
(errcode(ERRCODE_SYNTAX_ERROR),
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added here because contrib/babelfishpg_tsql/src/tsqlUnsupportedFeatureHandler.cpp has no check for Into() yet. Can be done through that also but will need more changes

@deepakshi-mittal deepakshi-mittal changed the title Added escape_hatch_identity_function for IDENTITY() in select into Added escape hatch for IDENTITY() in select into Nov 22, 2023
Need this because identity column will always be the last one in table
if created using a select into statement

Task: BABEL-539

Signed-off-by: Deepakshi Mittal <[email protected]>
Updated file BABEL-UNSUPPORTED, Added escape_hatch_identity_function

Task: BABEL-539

Signed-off-by: Deepakshi Mittal <[email protected]>
Copy link
Contributor

@forestkeeper forestkeeper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@forestkeeper forestkeeper merged commit 9237ce8 into babelfish-for-postgresql:BABEL_3_X_DEV Nov 22, 2023
29 checks passed
deepakshi-mittal added a commit to amazon-aurora/babelfish_extensions that referenced this pull request Nov 27, 2023
…resql#2055)

Need this because identity column will always be the last one in table
if created using a select into statement

Task: BABEL-539
Signed-off-by: Deepakshi Mittal <[email protected]>
(cherry picked from commit 9237ce8)
forestkeeper pushed a commit that referenced this pull request Nov 29, 2023
…ABLE (#2069)


Need this because identity column will always be the last one in table
if created using a select into statement

Task: BABEL-539
Signed-off-by: Deepakshi Mittal <[email protected]>
(cherry picked from commit 9237ce8)
Sairakan pushed a commit to amazon-aurora/babelfish_extensions that referenced this pull request Dec 21, 2023
…resql#2055)


Need this because identity column will always be the last one in table
if created using a select into statement

Task: BABEL-539
Signed-off-by: Deepakshi Mittal <[email protected]>
Sairakan pushed a commit to amazon-aurora/babelfish_extensions that referenced this pull request Dec 24, 2023
…resql#2055)


Need this because identity column will always be the last one in table
if created using a select into statement

Task: BABEL-539
Signed-off-by: Deepakshi Mittal <[email protected]>
Sairakan pushed a commit to amazon-aurora/babelfish_extensions that referenced this pull request Dec 24, 2023
…resql#2055)


Need this because identity column will always be the last one in table
if created using a select into statement

Task: BABEL-539
Signed-off-by: Deepakshi Mittal <[email protected]>
Sairakan pushed a commit to amazon-aurora/babelfish_extensions that referenced this pull request Dec 24, 2023
…resql#2055)


Need this because identity column will always be the last one in table
if created using a select into statement

Task: BABEL-539
Signed-off-by: Deepakshi Mittal <[email protected]>
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.

2 participants