Skip to content

Add SET STATEMENT RETURN TYPES#5379

Merged
chands10 merged 1 commit intobloomberg:mainfrom
chands10:c2
Sep 12, 2025
Merged

Add SET STATEMENT RETURN TYPES#5379
chands10 merged 1 commit intobloomberg:mainfrom
chands10:c2

Conversation

@chands10
Copy link
Copy Markdown
Contributor

@chands10 chands10 commented Sep 9, 2025

bb migration

Tests added later on

Copy link
Copy Markdown

@roborivers roborivers left a comment

Choose a reason for hiding this comment

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

Coding style check: Success ✓.
Smoke testing: Error ⚠.
Cbuild submission: Success ✓.
Regression testing: 5/627 tests failed ⚠.

The first 10 failing tests are:
truncatesc_offline_generated
sc_transactional_rowlocks_generated
cdb2_close
sc_downgrade
logfill

Copy link
Copy Markdown

@roborivers roborivers left a comment

Choose a reason for hiding this comment

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

Coding style check: Success ✓.
Smoke testing: Error ⚠.
Cbuild submission: Success ✓.
Regression testing: 2/626 tests failed ⚠.

The first 10 failing tests are:
skipscan
sc_downgrade

Copy link
Copy Markdown

@roborivers roborivers left a comment

Choose a reason for hiding this comment

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

Coding style check: Success ✓.
Smoke testing: Error ⚠.
Cbuild submission: Error ⚠.
Regression testing: 1/627 tests failed ⚠.

The first 10 failing tests are:
logfill_logput_window_generated

@chands10
Copy link
Copy Markdown
Contributor Author

/adhoc-test cdb2_close logfill skipscan

Copy link
Copy Markdown

@roborivers roborivers left a comment

Choose a reason for hiding this comment

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

Coding style check: Success ✓.
Smoke testing: Error ⚠.
Cbuild submission: Error ⚠.
Regression testing: 1/3 tests failed ⚠.

The first 10 failing tests are:
logfill

Copy link
Copy Markdown

@roborivers roborivers left a comment

Choose a reason for hiding this comment

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

Coding style check: Success ✓.
Smoke testing: Error ⚠.
Cbuild submission: Error ⚠.
Regression testing: Success ✓.

Copy link
Copy Markdown

@roborivers roborivers left a comment

Choose a reason for hiding this comment

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

Coding style check: Success ✓.
Smoke testing: Error ⚠.
Cbuild submission: Error ⚠.
Regression testing: 1/3 tests failed ⚠.

The first 10 failing tests are:
logfill

Copy link
Copy Markdown

@roborivers roborivers left a comment

Choose a reason for hiding this comment

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

Coding style check: Success ✓.
Smoke testing: Error ⚠.
Cbuild submission: Error ⚠.
Regression testing: 1/3 tests failed ⚠.

The first 10 failing tests are:
logfill

akshatsikarwar
akshatsikarwar previously approved these changes Sep 12, 2025
Signed-off-by: Salil Chandra <schandra107@bloomberg.net>
@chands10 chands10 merged commit 238c977 into bloomberg:main Sep 12, 2025
4 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.

3 participants