Skip to content

Conversation

@rishitha957
Copy link
Contributor

bf: force fetching blobs if there are check_constraints (bypass updCols optimization)

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

The first 10 failing tests are:
sc_downgrade
insert_lots_ssl_generated
insert_lots
reco-ddlk-sql

@rishitha957 rishitha957 force-pushed the json-upd branch 2 times, most recently from 389e425 to b4ce791 Compare January 14, 2026 19:29
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: Error ⚠.
Regression testing: Success ✓.

The first 10 failing tests are:
logfill_logput_window_generated
insert_lots_ssl_generated
insert_lots

@rishitha957 rishitha957 force-pushed the json-upd branch 2 times, most recently from ddaa8fd to d12e2b7 Compare January 27, 2026 04:28
db/record.c Outdated
blobnums[i] = i;
}

rc = ix_find_blobs_by_rrn_and_genid_tran(iq, trans, rrn, vgenid, numblobs, blobnums, blobsizes, bloboffs,
Copy link
Contributor

Choose a reason for hiding this comment

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

Can this be optimized to fetch only the missing blobs?

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:
jsonify
consumer_non_atomic_default_consumer_generated
insert_lots_ssl_generated
insert_lots
reco-ddlk-sql

@rishitha957 rishitha957 force-pushed the json-upd branch 2 times, most recently from 2383557 to b41af60 Compare January 29, 2026 20:54
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: Error ⚠.
Regression testing: Success ✓.

The first 10 failing tests are:
sc_downgrade [setup failure]
sc_redo [core dumped]
logfill
jsonify
consumer_non_atomic_default_consumer_generated
insert_lots_ssl_generated
insert_lots

@rishitha957
Copy link
Contributor Author

/adhoc-test jsonify

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:
jsonify

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

…ptimization)

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.

Coding style check: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 0/0 tests failed ⚠.

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