Skip to content

Conversation

@farost
Copy link
Member

@farost farost commented Nov 24, 2025

Release notes: usage and product changes

Introduce protocol changes required by https://github.com/typedb/typedb-cluster/pull/642 :

  • User management requests now contain specific request information like username and password hashes (not actual passwords) instead of database commits.
  • Commit requests now contain commit profile-related flags.

Implementation

@farost farost requested a review from lolski November 24, 2025 16:15
@farost farost changed the title Refine raft requests for user management Complete replication requests for user management and commits Nov 25, 2025
@farost farost marked this pull request as ready for review November 25, 2025 18:13
message SchemaCommit {
string name = 1;
bytes commit_record = 2;
bool profile_enabled = 3;
Copy link
Member

Choose a reason for hiding this comment

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

in core, how do we make commit profile to be collected? do we configure a profiling flag in the CLI at bootup time?

Copy link
Member Author

Choose a reason for hiding this comment

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

We only commit from transactions, and, in transactions, their profiles are configured using tracing::enabled!(Level::TRACE)) <-- so it depends on the logging level.

Copy link
Member

Choose a reason for hiding this comment

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

what happens if an admin creates a three node cluster, all with differing log levels? i'm not sure how, but it feels like something that we have to prevent right?

to highlight the meta theme: all servers in a cluster should be configured identically.

Copy link
Member Author

Choose a reason for hiding this comment

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

I don't think they have to. We return the commit profile only of a single local execution, on the target node only. It's described in the server prs. I see no issue with different configurations. What do you see as an issue?

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