Skip to content

fix(core,cli,jdbc,webserver): compiler-warning fixes without suppressions#15503

Merged
AcevedoR merged 10 commits intodevelopfrom
fix/compiler-warnings-no-suppress
Apr 13, 2026
Merged

fix(core,cli,jdbc,webserver): compiler-warning fixes without suppressions#15503
AcevedoR merged 10 commits intodevelopfrom
fix/compiler-warnings-no-suppress

Conversation

@AcevedoR
Copy link
Copy Markdown
Contributor

@AcevedoR AcevedoR commented Apr 10, 2026

fix various compile warnings
I only selected the least impacting ones for this batch

@github-project-automation github-project-automation Bot moved this to To review in Pull Requests Apr 10, 2026
@AcevedoR AcevedoR force-pushed the fix/compiler-warnings-no-suppress branch from 2156f52 to 0cb2302 Compare April 10, 2026 15:26
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 10, 2026

Tests report quick summary:

success ✅ > tests: 4842, success: 4818, skipped: 22, failed: 2

unfold for details
Project Status Success Skipped Failed
cli success ✅ 47 0 0
core success ✅ 2315 0 0
executor success ✅ 38 0 0
jdbc success ✅ 11 0 0
jdbc-h2 success ✅ 564 0 0
jdbc-mysql success ✅ 554 0 0
jdbc-postgres success ✅ 559 0 0
processor success ✅ 7 0 0
queue success ✅ 31 0 0
runner-memory success ✅ 1 0 0
scheduler success ✅ 62 0 0
script success ✅ 31 0 0
storage-local success ✅ 64 0 0
webserver success ✅ 419 0 0
worker success ✅ 51 0 0
worker-controller success ✅ 86 0 0

Develocity build scan: https://develocity.kestra.io/s/flnuwova24iz6

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 10, 2026

🧪 Java Unit Tests

TestsPassed ✅Skipped ⚠️FailedTime ⏱
Java Tests Report4840 ran4818 ✅22 ⚠️0 ❌44m 30s 702ms

🐋 Docker image

ghcr.io/kestra-io/kestra-pr:15503
docker run --pull=always --rm -it -p 8080:8080 --user=root -v /var/run/docker.sock:/var/run/docker.sock -v /tmp:/tmp ghcr.io/kestra-io/kestra-pr:15503 server local

Comment thread core/src/main/java/io/kestra/core/docs/JsonSchemaCache.java Outdated
Comment thread core/src/main/java/io/kestra/core/queues/GenericQueueInterface.java Outdated
Comment thread executor/src/main/java/io/kestra/executor/DefaultExecutor.java
Comment thread webserver/src/main/java/io/kestra/webserver/controllers/api/FlowController.java Outdated
@AcevedoR AcevedoR force-pushed the fix/compiler-warnings-no-suppress branch from 5732831 to e86d2d5 Compare April 13, 2026 12:07
Copy link
Copy Markdown
Member

@loicmathieu loicmathieu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, nice cleaning

AcevedoR added 10 commits April 13, 2026 15:14
…ut suppressions

- serialVersionUID on 9 Serializable exception/inner classes
- transient on non-serializable fields in HttpClient exceptions
- @Builder.Default on 5 @SuperBuilder fields that had initializers ignored
- lombok.config: set addJacksonAnnotations=jackson2 to resolve ambiguity
- ObjectNode.fields() → properties() (Jackson 2.21 API)
- Artifact.getFile() → getPath() (Maven Resolver 2.x API)
- QueryFilter.asValues(): String[] widened to Object[] before Arrays.asList()
- CompletableFuture<?>[]: typed wildcard arrays replacing raw arrays
- PebbleEngineFactory: Class<?>[] for Proxy.newProxyInstance
- AbstractTrigger/Check/SseRequest/ChartFiltersOverrides: @Builder.Default
- MavenPluginDownloader: getPath() for artifact resolution
- PluginManager: default void close() — narrows AutoCloseable contract
- GenericQueueInterface: default void close() throws IOException
- HttpClientRequestException/ResponseException: transient fields
- Slugify: trim leading/trailing hyphens fix
- JsonSchemaCache: registerDefaultTypes() moved to @PostConstruct
- MonitoredWorkerQueue: made final (not subclassed)
- ServiceRegistry/all server commands/DefaultExecutor/Docker/
  ExecutionController: Await → Awaitility for production code
@AcevedoR AcevedoR force-pushed the fix/compiler-warnings-no-suppress branch from 3a55ecb to 921b66a Compare April 13, 2026 13:14
@AcevedoR AcevedoR merged commit 66937ba into develop Apr 13, 2026
10 of 12 checks passed
@AcevedoR AcevedoR deleted the fix/compiler-warnings-no-suppress branch April 13, 2026 14:28
@github-project-automation github-project-automation Bot moved this from To review to Done in Pull Requests Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants