You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We use proto QuerySessionInfo to maintain the query session information. However, status is changeable. So, it is not suitable to be kept inside the proto.
In this issue, we are going to remove status from QuerySessionInfo and will put status in SessionManager instead.
message QuerySessionInfo {
// the hex encoded string
int32 id = 1;
int64 start_time = 2;
SessionStatus status = 3;
int32 query_count = 4;
}
Remove the unstable status field and just keep it in SDK or Storage Node
Would you like to implement this feature? [y/n]
Solution
Alternatives
Additional context
The text was updated successfully, but these errors were encountered:
Motivations
We use proto
QuerySessionInfo
to maintain the query session information. However,status
is changeable. So, it is not suitable to be kept inside the proto.In this issue, we are going to remove
status
fromQuerySessionInfo
and will putstatus
in SessionManager instead.Remove the unstable status field and just keep it in SDK or Storage Node
Solution
Alternatives
Additional context
The text was updated successfully, but these errors were encountered: