Skip to content

Feature/last seen - #1901

Open
Arsenii-Malov wants to merge 3 commits into
mainfrom
feature/last-seen
Open

Feature/last seen#1901
Arsenii-Malov wants to merge 3 commits into
mainfrom
feature/last-seen

Conversation

@Arsenii-Malov

Copy link
Copy Markdown
Contributor

No description provided.

Arsenii-Malov and others added 3 commits August 24, 2026 18:08
An agent heartbeat for a device that is already ONLINE went through
MachineRepository.save(), which the MachineTagEventAspect turns into a
message on shared.devices-topic — roughly one per device per minute, and
the dominant reason the topic grew to ~8 GB inside the 72h retention.
lastSeen is not part of MachinePinotMessage, so those messages were
byte-for-byte identical to the previous one bar ingestionTime, and the
Pinot upsert table discards them anyway.

MachineStatusService now short-circuits when the status is unchanged and
refreshes lastSeen through a targeted Mongo update that bypasses save(),
so the aspect does not fire. Anything that touches a field carried in
MachinePinotMessage still goes through save()/saveAll().

CU-86ak2t1yn

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01J5H1THwzYNBUP3ahwkuYW5
@Arsenii-Malov
Arsenii-Malov requested a review from a team August 24, 2026 16:24
@Arsenii-Malov Arsenii-Malov self-assigned this Aug 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant