Skip to content

HCK-18365: fix boolean false in separate column DEFAULT statements - #64

Merged
taras-dubyk merged 1 commit into
developfrom
fix/boolean-default-false-separate-statement
Oct 30, 2026
Merged

taras-dubyk merged 1 commit into
developfrom
fix/boolean-default-false-separate-statement

Conversation

@Sylvie-P

@Sylvie-P Sylvie-P commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor
Sub-bugHCK-18365 [Cockroach] FE omits boolean default value false in separate ALTER statements

## Summary - Fix `assignTemplates` so boolean `false` is not treated as a missing placeholder when generating separate column DEFAULT statements. - [HCK-18365](https://hackolade.atlassian.net/browse/HCK-18365) (parent [HCK-18361](https://hackolade.atlassian.net/browse/HCK-18361)). - Same change as [hackolade/PostgreSQL#208](https://github.com/hackolade/PostgreSQL/pull/208).

Test plan

  • Boolean default false, column default values in separate statements
  • Default true, numeric defaults, inline mode unchanged

Made with Cursor

assignTemplates treated false as missing when filling placeholders, which broke SET DEFAULT false in separate FE statements.

Co-authored-by: Cursor <cursoragent@cursor.com>
@sonarqubecloud

Copy link
Copy Markdown

@Sylvie-P
Sylvie-P requested a review from taras-dubyk October 29, 2026 17:12
@taras-dubyk
taras-dubyk enabled auto-merge (squash) October 30, 2026 08:23
@taras-dubyk
taras-dubyk disabled auto-merge October 30, 2026 08:25
@taras-dubyk
taras-dubyk merged commit 42df96d into develop Oct 30, 2026
8 of 9 checks passed
@taras-dubyk
taras-dubyk deleted the fix/boolean-default-false-separate-statement branch October 30, 2026 08:25
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