From 7d23c8a232638a05e1a532dc6471b33acc503224 Mon Sep 17 00:00:00 2001 From: Matthew Lam Date: Mon, 9 Dec 2024 10:43:02 -0800 Subject: [PATCH 1/3] Update P4Runtime Spec to explicitly specify read-write symmetry exception for backup replicas. Signed-off-by: Matthew Lam --- docs/v1/P4Runtime-Spec.mdk | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/docs/v1/P4Runtime-Spec.mdk b/docs/v1/P4Runtime-Spec.mdk index c1124ed7..5e4c47c2 100755 --- a/docs/v1/P4Runtime-Spec.mdk +++ b/docs/v1/P4Runtime-Spec.mdk @@ -2297,7 +2297,9 @@ preserved, it will be explicitly stated in this document. The `MessageDifferencer` class [@ProtoMessageDifferencer] included in the Protobuf C++ API supports comparing messages while treating repeated fields as sets, so that different orderings of the same elements are considered equal. This method -of comparing Protobuf messages may come at a cost in performance. +of comparing Protobuf messages may come at a cost in performance. +The `backup_replicas` must be returned in order when read to preserve the +hierarchy of highest-preferred backup replicas. ### Data plane volatile objects { #sec-data-plane-volatile-objects } From 42d7d6d06db1007a46b94285f2cf7715fd691f48 Mon Sep 17 00:00:00 2001 From: Matthew Lam Date: Mon, 9 Dec 2024 10:48:20 -0800 Subject: [PATCH 2/3] Remove whitespace. Signed-off-by: Matthew Lam --- docs/v1/P4Runtime-Spec.mdk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/v1/P4Runtime-Spec.mdk b/docs/v1/P4Runtime-Spec.mdk index 5e4c47c2..631ab029 100755 --- a/docs/v1/P4Runtime-Spec.mdk +++ b/docs/v1/P4Runtime-Spec.mdk @@ -2297,7 +2297,7 @@ preserved, it will be explicitly stated in this document. The `MessageDifferencer` class [@ProtoMessageDifferencer] included in the Protobuf C++ API supports comparing messages while treating repeated fields as sets, so that different orderings of the same elements are considered equal. This method -of comparing Protobuf messages may come at a cost in performance. +of comparing Protobuf messages may come at a cost in performance. The `backup_replicas` must be returned in order when read to preserve the hierarchy of highest-preferred backup replicas. From d411ef03e693b8ce66bb470e3ff1306ee3d62a51 Mon Sep 17 00:00:00 2001 From: Matthew Lam Date: Tue, 17 Dec 2024 13:22:37 -0800 Subject: [PATCH 3/3] Update P4Runtime-Spec.mdk Signed-off-by: Matthew Lam --- docs/v1/P4Runtime-Spec.mdk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/v1/P4Runtime-Spec.mdk b/docs/v1/P4Runtime-Spec.mdk index 631ab029..f08938b1 100755 --- a/docs/v1/P4Runtime-Spec.mdk +++ b/docs/v1/P4Runtime-Spec.mdk @@ -2299,7 +2299,7 @@ C++ API supports comparing messages while treating repeated fields as sets, so that different orderings of the same elements are considered equal. This method of comparing Protobuf messages may come at a cost in performance. The `backup_replicas` must be returned in order when read to preserve the -hierarchy of highest-preferred backup replicas. +sequence of highest-preferred backup replicas. ### Data plane volatile objects { #sec-data-plane-volatile-objects }