Git-Mirrors/veilid
1
0
Fork 0
mirror of https://gitlab.com/veilid/veilid.git synced 2024-10-01 01:26:08 -04:00
Code Issues Packages Projects Releases Wiki Activity

clarify semantics

Browse Source
This commit is contained in:
Christien Rioux 2024-03-05 19:26:23 -05:00
parent 119a5668ac
commit 86f7473727
5 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
veilid-core/proto/veilid.capnp
View File

@ -353,7 +353,7 @@ struct OperationSetValueA @0x9378d0732dc95be2 {
struct OperationWatchValueQ @0xf9a5a6c547b9b228 {
key @0 :TypedKey; # key for value to watch
subkeys @1 :List(SubkeyRange); # subkey range to watch (up to 512 subranges), can not be empty, to watch everything use 0..=UINT32_MAX
subkeys @1 :List(SubkeyRange); # subkey range to watch (up to 512 subranges), if empty this implies 0..=UINT32_MAX
expiration @2 :UInt64; # requested timestamp when this watch will expire in usec since epoch (can be return less, 0 for max)
count @3 :UInt32; # requested number of changes to watch for (0 = cancel, 1 = single shot, 2+ = counter, UINT32_MAX = continuous)
watchId @4 :UInt64; # if 0, request a new watch. if >0, existing watch id
@ -370,7 +370,7 @@ struct OperationWatchValueA @0xa726cab7064ba893 {
struct OperationValueChanged @0xd1c59ebdd8cc1bf6 {
key @0 :TypedKey; # key for value that changed
subkeys @1 :List(SubkeyRange); # subkey range that changed (up to 512 ranges at a time, this can be empty if this is a watch expiration notice)
subkeys @1 :List(SubkeyRange); # subkey range that changed (up to 512 ranges at a time, if empty this is a watch expiration notice)
count @2 :UInt32; # remaining changes left (0 means watch has expired)
watchId @3 :UInt64; # watch id this value change came from
value @4 :SignedValueData; # first value that changed (the rest can be gotten with getvalue)

2
veilid-core/proto/veilid_capnp.rs
View File

@ -21406,4 +21406,4 @@ pub mod operation {
}
}
//BUILDHASH:2361d45ebb46feb1cecc71c1756fc90ff94487663e8d7211177e6df6e4033386
//BUILDHASH:4ee16918900e322a0daa4bb0a11ca6094989c65936a653b0b78811e027d4d962

2
veilid-core/src/rpc_processor/rpc_watch_value.rs
View File

@ -131,8 +131,6 @@ impl RPCProcessor {
// Validate accepted requests
if accepted {
xxx does this make sense?
// Verify returned answer watch id is the same as the question watch id if it exists
if let Some(question_watch_id) = question_watch_id {
if question_watch_id != watch_id {

3
veilid-core/src/storage_manager/mod.rs
View File

@ -566,8 +566,7 @@ impl StorageManager {
Ok(None)
}
/// Add or change an outbound watch to a DHT value
xxx decide if empty subkey range is valid. should probably reject that everywhere and use a default range of 'full'
/// Create,update or cancel an outbound watch to a DHT value
pub async fn watch_values(
&self,
key: TypedKey,

5
veilid-core/src/veilid_api/routing_context.rs
View File

@ -389,7 +389,10 @@ impl RoutingContext {
/// Cancels a watch early
///
/// This is a convenience function that cancels watching all subkeys in a range
/// This is a convenience function that cancels watching all subkeys in a range. The subkeys specified here
/// are subtracted from the watched subkey range. If no range is specified, this is equivalent to cancelling the entire range of subkeys.
/// Only the subkey range is changed, the expiration and count remain the same.
/// If no subkeys remain, the watch is entirely cancelled and will receive no more updates.
/// Returns Ok(true) if there is any remaining watch for this record
/// Returns Ok(false) if the entire watch has been cancelled
#[instrument(target = "veilid_api", level = "debug", ret, err)]