Skip to content

{182642483} keeping track of client identity in clientstats#5710

Merged
riverszhang89 merged 1 commit intobloomberg:mainfrom
riverszhang89:statclntid
Mar 6, 2026
Merged

{182642483} keeping track of client identity in clientstats#5710
riverszhang89 merged 1 commit intobloomberg:mainfrom
riverszhang89:statclntid

Conversation

@riverszhang89
Copy link
Copy Markdown
Contributor

/plugin-branch statclntid

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

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

The first 10 failing tests are:
cdb2api_2pc [setup core dumped]
cdb2_coltype [setup core dumped]
socksql_master_swings [core dumped]
consumer [core dumped]
insert_lots_ssl_generated
conn
consumer_non_atomic_default_consumer_generated
insert_lots
clientstats_leaks

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

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

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

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

@riverszhang89
Copy link
Copy Markdown
Contributor Author

cdb2test Feb 11 16:52:09 2026 success statclntid.R20260211.6

@riverszhang89 riverszhang89 force-pushed the statclntid branch 2 times, most recently from eec0a70 to 33e07b9 Compare February 23, 2026 18:21
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.

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

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

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.

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

The first 10 failing tests are:
sc_resume_logicalsc_generated
consumer_non_atomic_default_consumer_generated
sc_transactional_rowlocks_generated
remsql_locks
remsql_locks_rte_connect_generated
sc_truncate_lockorder_generated
reco-ddlk-sql

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.

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

The first 10 failing tests are:
sc_timepart
cldeadlock
consumer_non_atomic_default_consumer_generated
remsql_locks_rte_connect_generated
remsql_locks

Signed-off-by: Rivers Zhang <hzhang320@bloomberg.net>
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.

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

The first 10 failing tests are:
scindex_logicalsc_generated
disttxn
consumer_non_atomic_default_consumer_generated
remsql_locks_rte_connect_generated
remsql_locks
reco-ddlk-sql

Copy link
Copy Markdown
Contributor

@rishitha957 rishitha957 left a comment

Choose a reason for hiding this comment

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

Changes make sense to me!

@riverszhang89 riverszhang89 merged commit 0f66630 into bloomberg:main Mar 6, 2026
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