Skip to content

Conversation

infvg
Copy link

@infvg infvg commented Sep 1, 2025

No description provided.

@prestodb-ci prestodb-ci added the from:IBM PRs from IBM label Sep 1, 2025
1. Add an option of having catalogs stored remotely as opposed to locally.
2. Add a way to interact with the external store via Presto

### Dynamic Catalog Store
Copy link
Contributor

Choose a reason for hiding this comment

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

Some high level feedback -

  1. Let's call out what also happens at worker startup - With the dynamic catalog store, I think it makes sense to not have any sort of local disk based catalog info for any node. Instead, at startup the worker always gets (via a pull or a push by the RM) the most up-to-date catalog info over the wire

  2. We should flesh out how the coordinator decides to atomically switch to new (set of catalog configs). IMO this should be done via the heartbeat mechanism + consensus (say 2/3rd of the workers ack that they have new configs). However, I am not familiar with the heartbeat mechanism - specifically if we can make any latency and atomicity guarantees on it

  3. Even with the atomic config switchover, queries that are already in-progress can fail if their configs are over-written. To address this, we need a way to support multiple config-sets (at least two - the current and previous ones). We can associate each query with a catalog_config_id to address this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

from:IBM PRs from IBM

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants