Skip to content

Fixed error while using PosgreSQL reserved keyword in cursor operations#3866

Merged
shardgupta merged 7 commits intobabelfish-for-postgresql:BABEL_5_X_DEVfrom
amazon-aurora:curser-in-prepared-st
Jun 26, 2025
Merged

Fixed error while using PosgreSQL reserved keyword in cursor operations#3866
shardgupta merged 7 commits intobabelfish-for-postgresql:BABEL_5_X_DEVfrom
amazon-aurora:curser-in-prepared-st

Conversation

@shardgupta
Copy link
Copy Markdown
Contributor

@shardgupta shardgupta commented Jun 21, 2025

Description

During the cursor prepare, execution flow for statement is not passed through antlr parser. This causes issue when query contains PosgreSQL reserved keywords. This PR addresses this issue, along with any potential issue with query which is not compatible with PostgreSQL

Task: BABEL-5743

Signed-off-by: Shard Gupta shardga@amazon.com

Sharath BP and others added 2 commits June 21, 2025 13:35
@coveralls
Copy link
Copy Markdown
Collaborator

coveralls commented Jun 21, 2025

Pull Request Test Coverage Report for Build 15900333032

Details

  • 12 of 12 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.03%) to 75.857%

Totals Coverage Status
Change from base Build 15889588684: 0.03%
Covered Lines: 49453
Relevant Lines: 65192

💛 - Coveralls

@shardgupta shardgupta requested review from KushaalShroff and tanscorpio7 and removed request for KushaalShroff June 22, 2025 05:58
@shardgupta shardgupta force-pushed the curser-in-prepared-st branch from 37276b7 to 66ad5e1 Compare June 25, 2025 06:55
comments and code refactoring

Signed-off-by: Shard Gupta <shardga@amazon.com>
@shardgupta shardgupta force-pushed the curser-in-prepared-st branch from 66ad5e1 to 42a208d Compare June 25, 2025 06:56
@shardgupta shardgupta changed the title [DRAFT] Curser in prepared st Curser in prepared st Jun 25, 2025
Signed-off-by: Shard Gupta <shardga@amazon.com>
Signed-off-by: Shard Gupta <shardga@amazon.com>
Signed-off-by: Shard Gupta <shardga@amazon.com>
Signed-off-by: Shard Gupta <shardga@amazon.com>
@shardgupta shardgupta force-pushed the curser-in-prepared-st branch from 243622f to c9a2e21 Compare June 26, 2025 11:13
@shardgupta shardgupta changed the title Curser in prepared st Fixed error while using PosgreSQL reserved keyword in cursor operations Jun 26, 2025
@shardgupta shardgupta merged commit e6e094c into babelfish-for-postgresql:BABEL_5_X_DEV Jun 26, 2025
47 checks passed
@shardgupta shardgupta deleted the curser-in-prepared-st branch June 26, 2025 12:49
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.

4 participants