Skip to content

Conversation

@chands10
Copy link
Contributor

@chands10 chands10 commented Nov 6, 2025

/plugin-branch nomodefault

static inline void only_read_config(cdb2_hndl_tp *hndl, int *default_err)
{
read_available_comdb2db_configs(NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL);
if (default_err && strcasecmp(hndl->type, "default") == 0) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added that we only do this check if default_err is defined

Copy link
Contributor

Choose a reason for hiding this comment

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

👍

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

The first 10 failing tests are:
analyze
analyze_partial_index_off_generated
api_events
sc_transactional_rowlocks_generated
insert_lots_ssl_generated
insert_lots
reco-ddlk-sql

@chands10
Copy link
Contributor Author

chands10 commented Nov 7, 2025

cdb2test Nov 7 00:22:07 2025 success nomodefault.R20251106.13

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

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

The first 10 failing tests are:
logfill_logput_window_generated
consumer_non_atomic_default_consumer_generated
truncatesc_offline_generated
insert_lots
reco-ddlk-sql

@chands10 chands10 merged commit d9a2e37 into bloomberg:main Nov 21, 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