Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit a2c6ab2

Browse files
committedFeb 6, 2024
historical_uptime: clean up code
Signed-off-by: bingyuyap <bingyu.yap.21@gmail.com>
1 parent 33951b1 commit a2c6ab2

File tree

2 files changed

+4
-12
lines changed

2 files changed

+4
-12
lines changed
 

‎fly/pkg/db/index.go

+1-10
Original file line numberDiff line numberDiff line change
@@ -12,9 +12,6 @@ import (
1212
const (
1313
// Format for metricsChecked index key: metricsChecked|<bool>|<messageID>
1414
metricsCheckedIndexKeyFmt = "metricsChecked|%t|%s"
15-
// Format for lastObservedAt index key: lastObservedAt|<timestamp>|<messageID>
16-
// timestamp is in the format "20060102150405" (YYYYMMDDHHMMSS)
17-
lastObservedAtIndexKeyFmt = "lastObservedAt|%s|%s"
1815
)
1916

2017
// CreateOrUpdateIndex creates or updates indexes for a message
@@ -25,16 +22,10 @@ func CreateOrUpdateIndex(txn *badger.Txn, message *Message) error {
2522
return fmt.Errorf("failed to set metricsChecked index: %w", err)
2623
}
2724

28-
// Index for lastObservedAt
29-
loaKey := fmt.Sprintf(lastObservedAtIndexKeyFmt, message.LastObservedAt.Format("20060102150405"), message.MessageID)
30-
if err := txn.Set([]byte(loaKey), []byte(message.MessageID)); err != nil {
31-
return fmt.Errorf("failed to set lastObservedAt index: %w", err)
32-
}
33-
3425
return nil
3526
}
3627

37-
// Helper function to parse index key and extract values
28+
// parseMetricsCheckedIndexKey helper function to parse index key and extract values
3829
func parseMetricsCheckedIndexKey(key []byte) (bool, string, error) {
3930
keyStr := string(key) // Convert byte slice to string
4031
parts := strings.Split(keyStr, "|")

‎fly/pkg/db/message.go

+3-2
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,9 @@ import (
1414

1515
// Message represents the data structure for a message in the Observations table.
1616
type Message struct {
17-
MessageID MessageID `json:"messageId"`
17+
MessageID MessageID `json:"messageId"`
1818
LastObservedAt time.Time `json:"lastObservedAt"`
19-
Observations []Observation `json:"guardians"`
19+
Observations []Observation `json:"observations"`
2020
MetricsChecked bool `json:"metricsChecked"`
2121
}
2222

@@ -44,6 +44,7 @@ type ObservationStatus int
4444

4545
const (
4646
OnTime ObservationStatus = iota
47+
// Late indicates that the observation was made 30 hours after the last observed timestamp
4748
Late
4849
)
4950

0 commit comments

Comments
 (0)
Please sign in to comment.