Re: [PR] fix(ui): update computing unit status on /compute tab every second [texera]

["Yicong-Huang (via GitHub)" <gi...@apache.org>]

Yicong-Huang commented on code in PR #4612:
URL: https://github.com/apache/texera/pull/4612#discussion_r3338556696


##########
frontend/src/app/dashboard/component/user/user-computing-unit/user-computing-unit.component.ts:
##########
@@ -145,6 +146,12 @@ export class UserComputingUnitComponent implements OnInit {
         this.allComputingUnits = units;
         this.entries = units.map(u => new DashboardEntry(u));
       });
+
+    interval(1000)
+      .pipe(untilDestroyed(this))
+      .subscribe(() => {
+        this.computingUnitStatusService.refreshComputingUnitList();
+      });

Review Comment:
   We can do time interval for now. For the long term it's better to be event triggered. (In another way to think, you are right now creating an event every a few seconds)



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@texera.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org