Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhance cluster and websocket client #90

Merged
merged 6 commits into from
Nov 27, 2023

Conversation

4t145
Copy link
Collaborator

@4t145 4t145 commented Nov 25, 2023

  1. refactor websocket client
  2. rewrite logics for cluster publish and wait response
  3. add async set mode for tardis_static macro
tardis_static! {
    async set some_config: Value;
}

async fn init() {
    ...
    set_some_config(my_value)
}

async fn another_init() {
    let config = some_config().await;
    // will pending until the config have been setted.
}
  1. remove the default values for ui/spec/ path in web server config.
  2. add some methods for cache client.

// load_cache_nodes_info()
info!("[Tardis.Cluster] remove inactive node {remote:?} from cache");
cache_nodes.remove(&ClusterRemoteNodeKey::NodeId(remote.node_id));
// TODO

Check notice

Code scanning / devskim

A "TODO" or similar was left in source code, possibly indicating incomplete functionality Note

Suspicious comment
@gudaoxuri gudaoxuri merged commit c46e640 into ideal-world:main Nov 27, 2023
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.

2 participants