PG1204 commented on code in PR #8552:
URL: https://github.com/apache/texera/pull/8552#discussion_r4054594754
##########
frontend/src/app/workspace/types/execute-workflow.interface.ts:
##########
@@ -82,10 +82,11 @@ export interface OperatorStatistics
extends Readonly<{
aggregatedInputRowCount: number;
aggregatedInputSize?: number;
- inputPortMetrics: Record<string, number>;
+ /** Absent when the snapshot has no per-port information at all; `{}`
means every port was zero. */
+ inputPortMetrics?: Record<string, number>;
Review Comment:
All three fixed. The description now names the optional fields on
OperatorStatistics and the branch changeOperatorStatistics gained, and the
command includes joint-ui.service.spec.ts.
The table is rebuilt from an actual run rather than edited: 263 passing, 38
new, counts taken against the merge base. One number differs from yours, the
restore spec is at 16, not 14, since it gained the two takeUntil specs after
you wrote this.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
