Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(66)

Unified Diff: common/api/logdog_coordinator/logs/v1/state.proto

Issue 1863973002: LogDog: Update to archival V2. (Closed) Base URL: https://github.com/luci/luci-go@grpcutil-errors
Patch Set: Fix proto comment. Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: common/api/logdog_coordinator/logs/v1/state.proto
diff --git a/common/api/logdog_coordinator/logs/v1/state.proto b/common/api/logdog_coordinator/logs/v1/state.proto
index f916c9dc5e7ad21a5db8c1e79d1a24b8032f9ce0..c27e027afab5e2d8532939d90b6ca9a18efad7de 100644
--- a/common/api/logdog_coordinator/logs/v1/state.proto
+++ b/common/api/logdog_coordinator/logs/v1/state.proto
@@ -17,12 +17,10 @@ message LogStreamState {
// The time when the log stream was registered with the Coordinator.
google.protobuf.Timestamp created = 2;
- // The time when the log stream's state was last updated.
- google.protobuf.Timestamp updated = 3;
// The stream index of the log stream's terminal message. If the value is -1,
// the log is still streaming.
- int64 terminal_index = 4;
+ int64 terminal_index = 3;
// ArchiveInfo contains archive details for the log stream.
message ArchiveInfo {
@@ -36,13 +34,16 @@ message LogStreamState {
// If true, all log entries between 0 and terminal_index were archived. If
// false, this indicates that the log stream was not completely loaded into
// intermediate storage when the archival interval expired.
- bool whole = 4;
+ bool complete = 4;
+
+ // The number of log
+ int64 log_entry_count = 5;
}
// If non-nil, the log stream is archived, and this field contains archival
// details.
- ArchiveInfo archive = 5;
+ ArchiveInfo archive = 4;
// Indicates the purged state of a log. A log that has been purged is only
// acknowledged to administrative clients.
- bool purged = 6;
+ bool purged = 5;
}
« no previous file with comments | « common/api/logdog_coordinator/logs/v1/pb.discovery.go ('k') | common/api/logdog_coordinator/logs/v1/state.pb.go » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698