Skip to content

Add 'index' to E6 reserved keywords#244

Merged
gauravdawar-e6 merged 1 commit intoe6data:mainfrom
tkaunlaky-e6:fix/e6-index-reserved-keyword
Apr 1, 2026
Merged

Add 'index' to E6 reserved keywords#244
gauravdawar-e6 merged 1 commit intoe6data:mainfrom
tkaunlaky-e6:fix/e6-index-reserved-keyword

Conversation

@tkaunlaky-e6
Copy link
Copy Markdown

Summary

  • Adds index to the E6 dialect's RESERVED_KEYWORDS set so it gets properly quoted when used as an identifier (CTE alias, column name, etc.)
  • Prevents conflicts with the INDEX keyword during transpilation

Test plan

  • Verified that quote_identifiers now quotes index in CTE definitions, CTE references, and column references when targeting the e6 dialect

Ensures that 'index' is quoted when used as an identifier (e.g. CTE
alias, column name) to avoid conflicts with the INDEX keyword.
Copy link
Copy Markdown
Collaborator

@gauravdawar-e6 gauravdawar-e6 left a comment

Choose a reason for hiding this comment

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

Looks good!

@gauravdawar-e6 gauravdawar-e6 merged commit 7eb42f0 into e6data:main Apr 1, 2026
6 checks passed
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