Skip to content

Conversation

heesung-sohn
Copy link
Contributor

Motivation

#24478 introduced tableview refresh logic when a metadata session is reestablished. For better debugging, we better print warn log in case the cache and tableview record are out-of-sync(not expected)

Modifications

  • added warn logs when the cache and tableview record in MetadataStoreTableViewImpl are out-of-sync
  • minor bug fix in the test code

Verifying this change

  • Make sure that the change passes the CI checks.

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

  • Dependencies (add or upgrade a dependency)
  • The public API
  • The schema
  • The default values of configurations
  • The threading model
  • The binary protocol
  • The REST endpoints
  • The admin CLI options
  • The metrics
  • Anything that affects deployment

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

Matching PR in forked repository

PR in forked repository:

…nd tableview are out of sync (ExtensibleLoadManagerImpl only)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-not-needed Your PR changes do not impact docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant