Skip to content

Commit 53b39cf

Browse files
committed
Consolidate watch logging
Signed-off-by: Brad Davidson <[email protected]>
1 parent 135886d commit 53b39cf

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

pkg/server/watch.go

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -157,11 +157,6 @@ func (w *watcher) Start(ctx context.Context, r *etcdserverpb.WatchCreateRequest)
157157
// get max revision from collected events
158158
if len(events) > 0 {
159159
revision = events[len(events)-1].KV.ModRevision
160-
if trace {
161-
for _, event := range events {
162-
logrus.Tracef("WATCH READ id=%d, key=%s, revision=%d", id, event.KV.Key, event.KV.ModRevision)
163-
}
164-
}
165160
}
166161

167162
// send response. note that there are no events if this is a progress response.
@@ -171,7 +166,13 @@ func (w *watcher) Start(ctx context.Context, r *etcdserverpb.WatchCreateRequest)
171166
WatchId: id,
172167
Events: toEvents(events...),
173168
}
174-
logrus.Tracef("WATCH SEND id=%d, key=%s, revision=%d, events=%d, size=%d, reads=%d", id, key, revision, len(wr.Events), wr.Size(), reads)
169+
if trace {
170+
keys := make([]string, len(wr.Events))
171+
for i, event := range wr.Events {
172+
keys[i] = string(event.Kv.Key)
173+
}
174+
logrus.Tracef("WATCH SEND id=%d, key=%s, revision=%d, events=%d, size=%d, reads=%d, keys=%s", id, key, revision, len(wr.Events), wr.Size(), reads, keys)
175+
}
175176
if err := w.server.Send(wr); err != nil {
176177
w.Cancel(id, 0, 0, err)
177178
}

0 commit comments

Comments
 (0)