Skip to content

HCK-13902: constraints as a separate statement#140

Merged
AlikRakhmonov merged 3 commits intodevelopfrom
fix/HCK-13902-constraints
Dec 17, 2025
Merged

HCK-13902: constraints as a separate statement#140
AlikRakhmonov merged 3 commits intodevelopfrom
fix/HCK-13902-constraints

Conversation

@chulanovskyi-bs
Copy link
Contributor

@chulanovskyi-bs chulanovskyi-bs commented Dec 17, 2025

Sub-bugHCK-13902 [PROD][Hive] Constraints are not handled properly

Content

  • properly combined and format all column constraints as a separate statements within CREATE TABLE clause. The only exception is DEFAULT which can be declared only as inline statement within column definition
image

@sonarqubecloud
Copy link

@AlikRakhmonov AlikRakhmonov enabled auto-merge (squash) December 17, 2025 16:37
@AlikRakhmonov AlikRakhmonov enabled auto-merge (squash) December 17, 2025 16:38
@AlikRakhmonov AlikRakhmonov merged commit 700616c into develop Dec 17, 2025
10 checks passed
@AlikRakhmonov AlikRakhmonov deleted the fix/HCK-13902-constraints branch December 17, 2025 16:55
Comment on lines +435 to +436
const value = typeof defaultValue === 'string' ? `'${defaultValue}'` : defaultValue;
result.inline = ` DEFAULT ${value}`;
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
const value = typeof defaultValue === 'string' ? `'${defaultValue}'` : defaultValue;
result.inline = ` DEFAULT ${value}`;
result.inline = ` DEFAULT ${defaultValue}`;

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