The GitHub Actions job "Build and push images" on texera.git/main has failed.
Run started by GitHub user bobbai00 (triggered by bobbai00).

Head commit for run:
6a9437ed6e46599044f4f7762dc23e4065624262 / Matthew B. <[email protected]>
refactor(frontend): standardize menu.component subscription cleanup (#5324)

### What changes were proposed in this PR?
- Removed the redundant `computingUnitStatusSubscription` field (and its
`.add()` wrapper and `ngOnDestroy` unsubscribe) in `menu.component.ts`:
the inner stream is already piped through `untilDestroyed(this)`, so the
manual aggregator did nothing.
- Replaced the manually managed `durationUpdateSubscription` with a
declarative `switchMap` pipe, so each `ExecutionDurationUpdateEvent`
cancels the previous 1s timer and starts a new one only while running,
instead of imperatively unsubscribing and reassigning a field.
- Dropped the now-unused `Subscription` import, leaving the component on
a single consistent cleanup style (`untilDestroyed`).
### Any related issues, documentation, or discussions?
Closes: #5323
### How was this PR tested?
- Ran `ng test
--include='src/app/workspace/component/menu/menu.component.spec.ts'`:
all 42 tests pass.
- Behavior-preserving refactor: same elapsed-timer cadence, same
base-duration updates, same teardown on destroy.
### Was this PR authored or co-authored using generative AI tooling?
Co-authored with Claude Opus 4.7 in compliance with ASF

Report URL: https://github.com/apache/texera/actions/runs/27050839578

With regards,
GitHub Actions via GitBox

Reply via email to