Skip to content

Comments

cleanup sending dummy byte, and use isnull instead#5719

Open
rishitha957 wants to merge 1 commit intobloomberg:mainfrom
rishitha957:rm_len1
Open

cleanup sending dummy byte, and use isnull instead#5719
rishitha957 wants to merge 1 commit intobloomberg:mainfrom
rishitha957:rm_len1

Conversation

@rishitha957
Copy link
Contributor

(backend new supports isnull flag, can remove sending 1 byte dummy data)

Copy link

@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 ✓.
Cbuild submission: Success ✓.
Regression testing: Success ✓.

The first 10 failing tests are:
consumer_non_atomic_default_consumer_generated
sc_transactional_rowlocks_generated
truncatesc_offline_generated
insert_lots_ssl_generated
reco-ddlk-sql

mohitkhullar
mohitkhullar previously approved these changes Feb 9, 2026
Copy link
Contributor

@mohitkhullar mohitkhullar left a comment

Choose a reason for hiding this comment

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

lgtm

…db2jdbc)

Signed-off-by: Rishitha Kalicheti <rkalicheti1@bloomberg.com>
Copy link

@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.

Cbuild submission: Error ⚠.
Regression testing: Success ✓.

The first 10 failing tests are:
consumer_non_atomic_default_consumer_generated
sc_transactional_rowlocks_generated
reco-ddlk-sql

Copy link

@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.

Cbuild submission: Error ⚠.
Regression testing: Success ✓.

The first 10 failing tests are:
maxtable
sc_resume_logicalsc_generated
consumer_non_atomic_default_consumer_generated
sc_timepart
reco-ddlk-sql

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