och5351 commented on code in PR #27874:
URL: https://github.com/apache/flink/pull/27874#discussion_r3025385511
##########
flink-runtime-web/web-dashboard/src/app/pages/job/rescales/job-rescales.component.html:
##########
@@ -22,6 +22,319 @@
[nzAnimated]="{ inkBar: true, tabPane: false }"
[nzTabBarExtraContent]="extraTemplate"
>
+ <nz-tab nzTitle="History">
+ <nz-table
+ class="no-border small"
+ [nzSize]="'small'"
+ [nzData]="rescalesHistories || []"
+ [nzFrontPagination]="false"
+ [nzShowPagination]="false"
+ >
+ <thead>
+ <tr>
+ <th nzWidth="70px" nzAlign="center"></th>
+ <th
+ nz-tooltip
+ nzTooltipTitle="The unique ID in Rescale consists of 32
hexadecimal characters"
+ >
+ <strong>Rescale UUID</strong>
+ </th>
+ <th
+ nz-tooltip
+ nzTooltipTitle="The number ID of Rescale attempts that occurred
under the same resource requirements"
+ >
+ <strong>Attempt ID</strong>
+ </th>
+ <th
+ nz-tooltip
+ nzTooltipTitle="The unique ID of resource requirements consists of
32 hexadecimal characters"
+ >
+ <strong>Requirements ID</strong>
+ </th>
+ <th nz-tooltip nzTooltipTitle="The reason that triggers the target
Rescale">
+ <strong>Trigger Cause</strong>
+ </th>
+ <th nz-tooltip nzTooltipTitle="The end state of the target Rescale">
+ <strong>Terminal State</strong>
+ </th>
+ <th
+ nz-tooltip
+ nzTooltipTitle="The reason for the completion or termination of
the target Rescale"
+ >
+ <strong>Terminated Reason</strong>
+ </th>
+ <th nz-tooltip nzTooltipTitle="The start time of the target Rescale">
+ <strong>Start Time</strong>
+ </th>
+ <th
+ nz-tooltip
+ nzTooltipTitle="Duration from the start of the rescale to its
completion or until now"
+ >
+ <strong>Duration</strong>
+ </th>
+ <th nz-tooltip nzTooltipTitle="The end time of the target Rescale">
+ <strong>End Time</strong>
+ </th>
+ </tr>
+ </thead>
+ <tbody>
+ <ng-container *ngFor="let history of rescalesHistories; trackBy:
trackById">
+ <tr>
+ <td
+ nzShowExpand
+ [(nzExpand)]="history.expand"
+ (nzExpandChange)="onExpandChange(history, $event)"
+ ></td>
+ <td nz-tooltip [nzTooltipTitle]="history.rescaleUuid">
+ {{ truncateUuid(history.rescaleUuid) }}
+ </td>
+ <td>{{ history.rescaleAttemptId }}</td>
+ <td nz-tooltip [nzTooltipTitle]="history.resourceRequirementsUuid">
+ {{ truncateUuid(history.resourceRequirementsUuid) }}
+ </td>
+ <td>{{ history.triggerCause }}</td>
+ <td><flink-job-badge
[state]="history.terminalState"></flink-job-badge></td>
+ <td>{{ history.terminatedReason }}</td>
+ <td>{{ history.startTimestampInMillis | date: 'yyyy-MM-dd
HH:mm:ss.SSS' }}</td>
+ <td>
+ {{ history.endTimestampInMillis - history.startTimestampInMillis
| humanizeDuration }}
+ </td>
+ <td>{{ history.endTimestampInMillis | date: 'yyyy-MM-dd
HH:mm:ss.SSS' }}</td>
+ </tr>
+ <tr [nzExpand]="history.expand">
+ <td colspan="10" *ngIf="history.expand" class="collapse-td">
+ <div class="rescale-details-wrapper">
+ <div class="rescale-details-box">
+ <h3><strong>Rescale Details</strong></h3>
+ <p>
+ Rescale UUID:
+ {{ truncateUuid(history.rescaleUuid) }}
+ Attempt ID:
+ {{ history.rescaleAttemptId }}
+
Requirements ID:
+ {{ truncateUuid(history.resourceRequirementsUuid) }}
+ Trigger
Cause:
+ {{ history.triggerCause }}
+ Terminal State:
+ {{ history.terminalState }}
+ Terminal Reason: {{ history.terminatedReason }}
+ </p>
Review Comment:
Thank you for the hint :)
For reference, this is just my personal opinion, but it seems like the
information regarding 'RescalesDetails' overlaps with the information in the
parent component.
What do you think about this duplication of information?
--
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]
