Skip to content

fdb: remove class_override; resolve fdb class from gbl tunables#6032

Open
dorinhogea wants to merge 1 commit into
bloomberg:mainfrom
dorinhogea:fdb-remove-class-override
Open

fdb: remove class_override; resolve fdb class from gbl tunables#6032
dorinhogea wants to merge 1 commit into
bloomberg:mainfrom
dorinhogea:fdb-remove-class-override

Conversation

@dorinhogea

Copy link
Copy Markdown
Contributor

Class/local are set at fdb creation from foreign_db_resolve_local and foreign_db_resolve_tier tunables. Per-statement _ prefix overrides are deprecated and ignored with a rate-limited warning.

This simplifies greatly the fdb sqlite interactions; this is a pre-requirement for tracking remote stats version in fdb core.

Class/local are set at fdb creation from foreign_db_resolve_local and
foreign_db_resolve_tier tunables. Per-statement _ prefix overrides are
deprecated and ignored with a rate-limited warning.

Signed-off-by: Dorin Hogea <dhogea@bloomberg.net>

@roborivers roborivers left a comment

Copy link
Copy Markdown

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_resume_logicalsc_generated **quarantined**
fdb_compare_rte_connect_generated
fdb_compare
sql_logfill_autodisable
consumer_non_atomic_default_consumer_generated **quarantined**
remsql_rte_connect_generated
remtran_origin
remsql_locks_rte_connect_generated
remsql_locks
remsql_fdbpushredirect_generated

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.

2 participants