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

Generate table properties if DDL contain default values #1387

Open
wants to merge 4 commits into
base: feature/teradata_support
Choose a base branch
from

Conversation

sriram251-code
Copy link
Contributor

closes #1384

This PR introduces a feature to generate table properties when a DDL includes default values.

@sriram251-code sriram251-code added the enhancement New feature or request label Dec 23, 2024
@sriram251-code sriram251-code self-assigned this Dec 23, 2024
@sriram251-code sriram251-code marked this pull request as ready for review December 23, 2024 13:58
@sriram251-code sriram251-code requested a review from a team as a code owner December 23, 2024 13:58
Copy link

github-actions bot commented Dec 23, 2024

Coverage tests results

506 tests  ±0   468 ✅ ±0   5s ⏱️ ±0s
  7 suites ±0    38 💤 ±0 
  7 files   ±0     0 ❌ ±0 

Results for commit 8b7bb16. ± Comparison against base commit 4976207.

♻️ This comment has been updated with latest results.

@sundarshankar89
Copy link
Collaborator

@sriram251-code this change is agnostic to all dialects as it is in generation side, so this needs changes in both ANLTR as well as other dialect functional tests.

@sriram251-code
Copy link
Contributor Author

@sundarshankar89 implemented in antlr and sqlglot please review.

@sundarshankar89
Copy link
Collaborator

@sundarshankar89 implemented in antlr and sqlglot please review.
Make this a separate PR and another PR for adding just TD capability.

Copy link
Collaborator

@sundarshankar89 sundarshankar89 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 small nit:

Copy link
Contributor

@ericvergnaud ericvergnaud left a comment

Choose a reason for hiding this comment

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

LGTM

@sundarshankar89
Copy link
Collaborator

@sriram251-code Can you recreate this PR.

@sundarshankar89
Copy link
Collaborator

@sriram251-code can this PR be closed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants