Task #89948

Simplify chunking of test-plan jobs

Added by Markus Klösges 5 months ago. Updated 5 months ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Tests
Target version:
-
Start date:
2019-12-14
Due date:
% Done:

100%

TYPO3 Version:
10
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

The current code ensuring a limited number of mssql jobs per stage is dealing with a massive amount of indices and is hard to follow.

A simplification could be provided, first splitting of the maximum number of mssql jobs per stage, and filling up stages other jobs are available.

Associated revisions

Revision 15904ee2 (diff)
Added by Markus Klösges 5 months ago

[TASK] Simplify chunking of test-plan jobs

To make the code for splitting all test-jobs into stages of equal size
simpler and easier to maintain, a helper class is introduced
that supports chunking two lists into equally sized chunks, while
ensuring that each chunk only contains a limited number of elements
from one of the lists.

Using this utility, the number of mssql jobs can easily be limited,
while keeping the spec-code easy to follow.

Releases: master, 9.5, 8.7
Resolves: #89948
Change-Id: I9e1677e19fb2ea6df7d9e964e8f772ffb7324ec8
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/62640
Tested-by: TYPO3com <>
Tested-by: Andreas Fernandez <>
Tested-by: Anja Leichsenring <>
Reviewed-by: Andreas Fernandez <>
Reviewed-by: Anja Leichsenring <>

Revision 3b7556a4 (diff)
Added by Markus Klösges 5 months ago

[TASK] Simplify chunking of test-plan jobs

To make the code for splitting all test-jobs into stages of equal size
simpler and easier to maintain, a helper class is introduced
that supports chunking two lists into equally sized chunks, while
ensuring that each chunk only contains a limited number of elements
from one of the lists.

Using this utility, the number of mssql jobs can easily be limited,
while keeping the spec-code easy to follow.

Releases: master, 9.5, 8.7
Resolves: #89948
Change-Id: I9e1677e19fb2ea6df7d9e964e8f772ffb7324ec8
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/62649
Tested-by: TYPO3com <>
Tested-by: Anja Leichsenring <>
Reviewed-by: Anja Leichsenring <>

Revision 6753a7dd (diff)
Added by Markus Klösges 5 months ago

[TASK] Simplify chunking of test-plan jobs

To make the code for splitting all test-jobs into stages of equal size
simpler and easier to maintain, a helper class is introduced
that supports chunking two lists into equally sized chunks, while
ensuring that each chunk only contains a limited number of elements
from one of the lists.

Using this utility, the number of mssql jobs can easily be limited,
while keeping the spec-code easy to follow.

Releases: master, 9.5, 8.7
Resolves: #89948
Change-Id: I9e1677e19fb2ea6df7d9e964e8f772ffb7324ec8
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/62673
Tested-by: TYPO3com <>
Tested-by: Anja Leichsenring <>
Reviewed-by: Anja Leichsenring <>

History

#1 Updated by Gerrit Code Review 5 months ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62640

#2 Updated by Gerrit Code Review 5 months ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62640

#3 Updated by Gerrit Code Review 5 months ago

Patch set 1 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62649

#4 Updated by Anonymous 5 months ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#5 Updated by Gerrit Code Review 5 months ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62673

#6 Updated by Anonymous 5 months ago

  • Status changed from Under Review to Resolved

#7 Updated by Benni Mack 5 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF