Skip to content

{180374805} No timeout for successfully authenticated connections#5355

Merged
mohitkhullar merged 1 commit intobloomberg:mainfrom
mohitkhullar:allow_old_authn
Sep 16, 2025
Merged

{180374805} No timeout for successfully authenticated connections#5355
mohitkhullar merged 1 commit intobloomberg:mainfrom
mohitkhullar:allow_old_authn

Conversation

@mohitkhullar
Copy link
Contributor

/plugin-branch allow_old_authn

@mohitkhullar
Copy link
Contributor Author

/plugin-branch allow_old_authn

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

The first 10 failing tests are:
tunables
snapshot_during_truncate

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

The first 10 failing tests are:
tunables

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

The first 10 failing tests are:
sc_transactional_rowlocks_generated

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

The first 10 failing tests are:
sc_transactional_rowlocks_generated

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

The first 10 failing tests are:
limitsortingtbl
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.

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

The first 10 failing tests are:
sc_transactional_rowlocks_generated

Signed-off-by: mohitkhullar <mkhullar1@bloomberg.net>
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. ⚠.
Smoke testing: Error ⚠.
Cbuild submission: Error ⚠.
Regression testing: 1/627 tests failed ⚠.

The first 10 failing tests are:
logfill_logput_window_generated

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

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

The first 10 failing tests are:
phys_rep_tiered
incoherent_startup
snapshot_during_truncate

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

The first 10 failing tests are:
logfill_logput_window_generated

mponomar
mponomar previously approved these changes Sep 15, 2025
mponomar
mponomar previously approved these changes Sep 15, 2025
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. ⚠.
Smoke testing: Error ⚠.
Cbuild submission: Success ✓.
Regression testing: 6/628 tests failed ⚠.

The first 10 failing tests are:
incoherent_slow
simple_timepart_reptimeout_generated
simple_timepart
incoh_remsql_rte_connect_generated
incoh_remsql
sc_addfield_logicalsc_generated

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

The first 10 failing tests are:
sc_transactional_rowlocks_generated

@mohitkhullar mohitkhullar merged commit 3293fb9 into bloomberg:main Sep 16, 2025
5 of 7 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