mudhorn/n8n
1
0
Fork 0
mirror of https://github.com/n8n-io/n8n.git synced 2025-02-21 02:56:40 -08:00
Code Issues Actions Packages Projects Releases Wiki Activity

fix(editor): Execution list micro optimization (#5244)

Browse source 
* fix(editor): Execution list micro optimization

* fix(editor): remove old variable usage
This commit is contained in:
Csaba Tuncsik 2023-01-25 22:01:06 +01:00 committed by GitHub
parent c711c53ad6
commit a1710fbd27
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 18 additions and 22 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

32
packages/editor-ui/src/components/ExecutionsList.vue
View file

@ -554,12 +554,7 @@ export default mixins(externalHooks, genericHelpers, executionHelpers, restApi,
}
},
getWorkflowName(workflowId: string): string | undefined {
const workflow = this.workflows.find((data) => data.id === workflowId);
if (workflow === undefined) {
return undefined;
}
return workflow.name;
return this.workflows.find((data) => data.id === workflowId)?.name;
},
async loadActiveExecutions(): Promise<void> {
const activeExecutions = await this.restApi().getCurrentExecutions(
@ -585,7 +580,7 @@ export default mixins(externalHooks, genericHelpers, executionHelpers, restApi,
// iF you use firstId, filtering id >= 504 you won't
// ever get ids 500, 501, 502 and 503 when they finish
const pastExecutionsPromise: Promise<IExecutionsListResponse> =
this.restApi().getPastExecutions(filter, 30);
this.restApi().getPastExecutions(filter, this.requestItemsPerRequest);
const currentExecutionsPromise: Promise<IExecutionsCurrentSummaryExtended[]> =
this.restApi().getCurrentExecutions({});
@ -603,7 +598,8 @@ export default mixins(externalHooks, genericHelpers, executionHelpers, restApi,
this.workflowsStore.activeExecutions = results[1];
// execution IDs are typed as string, int conversion is necessary so we can order.
const alreadyPresentExecutionIds = this.finishedExecutions.map((exec) =>
const alreadyPresentExecutions = [...this.finishedExecutions];
const alreadyPresentExecutionIds = alreadyPresentExecutions.map((exec) =>
parseInt(exec.id, 10),
);
let lastId = 0;
@ -611,7 +607,7 @@ export default mixins(externalHooks, genericHelpers, executionHelpers, restApi,
for (let i = results[0].results.length - 1; i >= 0; i--) {
const currentItem = results[0].results[i];
const currentId = parseInt(currentItem.id, 10);
if (lastId !== 0 && isNaN(currentId) === false) {
if (lastId !== 0 && !isNaN(currentId)) {
// We are doing this iteration to detect possible gaps.
// The gaps are used to remove executions that finished
// and were deleted from database but were displaying
@ -631,14 +627,14 @@ export default mixins(externalHooks, genericHelpers, executionHelpers, restApi,
// Execution that we received is already present.
if (
this.finishedExecutions[executionIndex].finished === false &&
alreadyPresentExecutions[executionIndex].finished === false &&
currentItem.finished === true
) {
// Concurrency stuff. This might happen if the execution finishes
// prior to saving all information to database. Somewhat rare but
// With auto refresh and several executions, it happens sometimes.
// So we replace the execution data so it displays correctly.
this.finishedExecutions[executionIndex] = currentItem;
alreadyPresentExecutions[executionIndex] = currentItem;
}
continue;
@ -646,23 +642,25 @@ export default mixins(externalHooks, genericHelpers, executionHelpers, restApi,
// Find the correct position to place this newcomer
let j;
for (j = this.finishedExecutions.length - 1; j >= 0; j--) {
if (currentId < parseInt(this.finishedExecutions[j].id, 10)) {
this.finishedExecutions.splice(j + 1, 0, currentItem);
for (j = alreadyPresentExecutions.length - 1; j >= 0; j--) {
if (currentId < parseInt(alreadyPresentExecutions[j].id, 10)) {
alreadyPresentExecutions.splice(j + 1, 0, currentItem);
break;
}
}
if (j === -1) {
this.finishedExecutions.unshift(currentItem);
alreadyPresentExecutions.unshift(currentItem);
}
}
this.finishedExecutions = this.finishedExecutions.filter(
const alreadyPresentExecutionsFiltered = alreadyPresentExecutions.filter(
(execution) =>
!gaps.includes(parseInt(execution.id, 10)) && lastId >= parseInt(execution.id, 10),
);
this.finishedExecutionsCount = results[0].count;
this.finishedExecutionsCountEstimated = results[0].estimated;
this.workflowsStore.addToCurrentExecutions(this.finishedExecutions);
Vue.set(this, 'finishedExecutions', alreadyPresentExecutionsFiltered);
this.workflowsStore.addToCurrentExecutions(alreadyPresentExecutionsFiltered);
},
async loadFinishedExecutions(): Promise<void> {
if (this.filter.status === 'running') {

8
packages/editor-ui/src/components/ExecutionsView/ExecutionsList.vue
View file

@ -326,7 +326,7 @@ export default mixins(
for (let i = fetchedExecutions.length - 1; i >= 0; i--) {
const currentItem = fetchedExecutions[i];
const currentId = parseInt(currentItem.id, 10);
if (lastId !== 0 && isNaN(currentId) === false) {
if (lastId !== 0 && !isNaN(currentId)) {
if (currentId - lastId > 1) {
const range = _range(lastId + 1, currentId);
gaps.push(...range);
@ -372,10 +372,8 @@ export default mixins(
if (updatedActiveExecution !== null) {
this.workflowsStore.activeWorkflowExecution = updatedActiveExecution;
} else {
const activeNotInTheList = !existingExecutions.some(
(ex) => ex.id === this.activeExecution?.id,
);
if (activeNotInTheList && this.executions.length > 0) {
const activeInList = existingExecutions.some((ex) => ex.id === this.activeExecution.id);
if (!activeInList && this.executions.length > 0) {
this.$router
.push({
name: VIEWS.EXECUTION_PREVIEW,