Skip to content

Commit

Permalink
metrics: swap ready_to_upload with period_end in index
Browse files Browse the repository at this point in the history
It's redundant and this will make the logic simpler.

Signed-off-by: Matt Klein <[email protected]>
  • Loading branch information
mattklein123 committed Oct 18, 2024
1 parent b5bfeb2 commit a0fe79a
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions src/bitdrift_public/protobuf/client/v1/metric.proto
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,10 @@ message PendingAggregationIndex {
string name = 1;
// The start of the period that this file covers. All metrics are aggregated over this period.
google.protobuf.Timestamp period_start = 2;
// True if an aggregation is ready to upload and should no longer be written to.
bool ready_to_upload = 3;
// When the aggregation was closed. If specified, the server can decide to handle variable
// size aggregation windows by averaging the data over the period or some other heuristic. Also
// indicates that the file is ready for upload.
google.protobuf.Timestamp period_end = 3;
}
// List of files, in order of period_start, that are pending upload.
repeated PendingFile pending_files = 1;
Expand Down

0 comments on commit a0fe79a

Please sign in to comment.