GRUD-DEV-1203/parallel thumbnail creation

[smnhgn]

No description provided.

[github-actions]

File	Coverage
All files	87%	✅
com/campudus/tableaux/exceptions.scala	0%	❌
com/campudus/tableaux/exceptions.scala	0%	❌
com/campudus/tableaux/arguments.scala	97%	✅
com/campudus/tableaux/Starter.scala	76%	❌
com/campudus/tableaux/controller/SystemController.scala	99%	✅
com/campudus/tableaux/controller/TableauxController.scala	98%	✅
com/campudus/tableaux/controller/UserController.scala	72%	❌
com/campudus/tableaux/controller/StructureController.scala	91%	✅
com/campudus/tableaux/controller/MediaController.scala	79%	❌
com/campudus/tableaux/database/historyType.scala	7%	❌
com/campudus/tableaux/database/languageType.scala	86%	✅
com/campudus/tableaux/database/dbType.scala	93%	✅
com/campudus/tableaux/database/historyType.scala	66%	❌
com/campudus/tableaux/database/database.scala	96%	✅
com/campudus/tableaux/database/database.scala	81%	✅
com/campudus/tableaux/database/database.scala	97%	✅
com/campudus/tableaux/database/domain/column.scala	71%	❌
com/campudus/tableaux/database/domain/column.scala	64%	❌
com/campudus/tableaux/database/domain/column.scala	95%	✅
com/campudus/tableaux/database/domain/table.scala	96%	✅
com/campudus/tableaux/database/domain/service.scala	81%	✅
com/campudus/tableaux/database/domain/userSetting.scala	60%	❌
com/campudus/tableaux/database/domain/column.scala	77%	❌
com/campudus/tableaux/database/domain/annotation.scala	14%	❌
com/campudus/tableaux/database/domain/column.scala	48%	❌
com/campudus/tableaux/database/domain/annotation.scala	60%	❌
com/campudus/tableaux/database/domain/history.scala	84%	✅
com/campudus/tableaux/database/domain/column.scala	0%	❌
com/campudus/tableaux/database/domain/annotation.scala	83%	✅
com/campudus/tableaux/database/domain/userSetting.scala	95%	✅
com/campudus/tableaux/database/domain/annotation.scala	80%	✅
com/campudus/tableaux/database/domain/userSetting.scala	94%	✅
com/campudus/tableaux/database/domain/table.scala	98%	✅
com/campudus/tableaux/database/domain/column.scala	0%	❌
com/campudus/tableaux/database/domain/column.scala	95%	✅
com/campudus/tableaux/database/domain/history.scala	60%	❌
com/campudus/tableaux/database/domain/file.scala	72%	❌
com/campudus/tableaux/database/domain/userSetting.scala	90%	✅
com/campudus/tableaux/database/domain/userSetting.scala	0%	❌
com/campudus/tableaux/database/domain/column.scala	90%	✅
com/campudus/tableaux/database/domain/domainobject.scala	99%	✅
com/campudus/tableaux/database/domain/column.scala	92%	✅
com/campudus/tableaux/database/domain/column.scala	78%	❌
com/campudus/tableaux/database/domain/domainobject.scala	93%	✅
com/campudus/tableaux/database/domain/column.scala	89%	✅
com/campudus/tableaux/database/model/HistoryModel.scala	98%	✅
com/campudus/tableaux/database/model/FolderModel.scala	95%	✅
com/campudus/tableaux/database/model/UserModel.scala	99%	✅
com/campudus/tableaux/database/model/AttachmentModel.scala	82%	✅
com/campudus/tableaux/database/model/TableauxModel.scala	91%	✅
com/campudus/tableaux/database/model/StructureModel.scala	95%	✅
com/campudus/tableaux/database/model/SystemModel.scala	85%	✅
com/campudus/tableaux/database/model/FileModel.scala	94%	✅
com/campudus/tableaux/database/model/structure/ColumnModel.scala	87%	✅
com/campudus/tableaux/database/model/structure/TableGroupModel.scala	79%	❌
com/campudus/tableaux/database/model/structure/ColumnModel.scala	88%	✅
com/campudus/tableaux/database/model/structure/TableModel.scala	85%	✅
com/campudus/tableaux/database/model/tableaux/RowModel.scala	87%	✅
com/campudus/tableaux/database/model/tableaux/RowModel.scala	93%	✅
com/campudus/tableaux/database/model/tableaux/RowModel.scala	89%	✅
com/campudus/tableaux/helper/FileUtils.scala	74%	❌
com/campudus/tableaux/helper/JsonUtils.scala	94%	✅
com/campudus/tableaux/helper/ResultChecker.scala	98%	✅
com/campudus/tableaux/helper/IdentifierFlattener.scala	83%	✅
com/campudus/tableaux/router/DocumentationRouter.scala	21%	❌
com/campudus/tableaux/router/BaseRouter.scala	64%	❌
com/campudus/tableaux/router/SystemRouter.scala	90%	✅
com/campudus/tableaux/router/MediaRouter.scala	78%	❌
com/campudus/tableaux/router/TableauxRouter.scala	82%	✅
com/campudus/tableaux/router/StructureRouter.scala	99%	✅
com/campudus/tableaux/router/RouterRegistry.scala	53%	❌
com/campudus/tableaux/router/auth/KeycloakAuthHandler.scala	72%	❌
com/campudus/tableaux/router/auth/permission/Condition.scala	89%	✅
com/campudus/tableaux/router/auth/permission/Condition.scala	74%	❌
com/campudus/tableaux/router/auth/permission/Condition.scala	87%	✅
com/campudus/tableaux/router/auth/permission/RoleModel.scala	87%	✅
com/campudus/tableaux/router/auth/permission/RoleModel.scala	0%	❌
com/campudus/tableaux/router/auth/permission/Condition.scala	90%	✅
com/campudus/tableaux/router/auth/permission/Condition.scala	79%	❌
com/campudus/tableaux/router/auth/permission/Action.scala	97%	✅
com/campudus/tableaux/router/auth/permission/Condition.scala	85%	✅
com/campudus/tableaux/router/auth/permission/TableauxUser.scala	63%	❌
com/campudus/tableaux/verticles/ThumbnailVerticle.scala	52%	❌
com/campudus/tableaux/verticles/EventClient.scala	72%	❌
com/campudus/tableaux/verticles/CacheVerticle.scala	76%	❌
com/campudus/tableaux/verticles/JsonSchemaValidator.scala	86%	✅
com/campudus/tableaux/verticles/MessagingVerticle.scala	83%	✅

Minimum allowed coverage is 80%

Generated by 🐒 cobertura-action against 697ecad

[github-actions]

Test Results

1 176 tests  ±0   1 168 ✅ ±0   4m 18s ⏱️ +41s
  146 suites ±0       8 💤 ±0 
  146 files   ±0       0 ❌ ±0 

Results for commit 697ecad. ± Comparison against base commit 049ab30.

♻️ This comment has been updated with latest results.

[zingmane]

Eine kleine Anmerkung um die mutable vars zu vermeiden.
Die anderen Anmerkungen sind in dem schon geschlossenen PR

[smnhgn]

Die maxExecuteTime vom worker pool habe ich vorerst auf 2 Minuten (wie im Vertx Beispiel) gelassen. Habt ihr da Vorschläge was wir da als sinnvollen Wert setzen sollen?

Ich würde hier eher die Zeit die wir vorher hatten und zu wenig war erhöhen. Der timeout ist ja pro Task. Also von den 2000ms auf z.b. 10_000ms gehen. Wenn das dann mal wirklcih länger dauert, finde ich es auch sinnvoll, dass ein waring im log steht. Mehrere logs dazu erwarte ich dazu eh nur im warump, im normalen Betrieb sollte das dann eigentlich nicht mehr vorkommen.

Laut Doku wird die Warning vom ThreadBlockChecker unabhängig von der maxExecuteTime immer nach 10s geloggt.
(When the blocking operation lasts more than the 10 seconds, a message will be printed on the console by the blocked thread checker. -> docs)

Ich habe mal getestet, was nach Ablauf der maxExecuteTime genau passiert, aber es scheint, dass einfach nur der Executor neu initialisiert wird (ohne warning oder error):

mMn. macht es gar keinen Sinn die maxExecutionTime überhaupt konfigurierbar zu machen.
Ich setze da jetzt mal fest nen Wert von 30 Sekunden.

[zingmane]

Finde ich so viel aufgeräumter und übersichtlicher 👍

[zingmane]

Ich sehe grade diesen PR. Ist das Thema nicht mehr aktuell, haben wir das nur vergessen? Wollte da noch jemand was testen?

@Zwergal
@McHunkyTrunk

[Zwergal]

Ich sehe grade diesen PR. Ist das Thema nicht mehr aktuell, haben wir das nur vergessen? Wollte da noch jemand was testen?

@Zwergal @McHunkyTrunk

Ich denke hier wurde nur der merge vergessen. Du hast ja approved.

[McHunkyTrunk]

Ich teste noch!