You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by "oscerd (via GitHub)" <gi...@apache.org> on 2026/03/31 10:35:30 UTC
[PR] Bump ai.docling:docling-serve-client from 0.4.7 to 0.5.0 [camel]
oscerd opened a new pull request, #22357:
URL: https://github.com/apache/camel/pull/22357
## Summary
_Claude Code on behalf of Andrea Cosentino_
- Bumps `ai.docling:docling-serve-client` from 0.4.7 to 0.5.0
- Adapts code to API breaking change: `ConvertDocumentResponse` is now a sealed abstract class with `InBodyConvertDocumentResponse` as the subclass containing the `getDocument()` method
- Uses `instanceof` pattern matching to handle the polymorphic response type in `DoclingProducer.extractDoclingDocument()` and `DoclingProducer.extractConvertedContent()`
- Updates unit test `DoclingAsyncConversionTest` to use `InBodyConvertDocumentResponse.builder()`
Supersedes #22139 (Dependabot PR with compilation errors).
## Test plan
- [x] Unit tests pass (46 tests, 0 failures)
- [x] Code formatted with `mvn formatter:format impsort:sort`
- [ ] CI integration tests (disabled on GitHub Actions due to resource constraints)
🤖 Generated with [Claude Code](https://claude.com/claude-code)
--
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: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Bump ai.docling:docling-serve-client from 0.4.7 to 0.5.0 [camel]
Posted by "oscerd (via GitHub)" <gi...@apache.org>.
oscerd commented on code in PR #22357:
URL: https://github.com/apache/camel/pull/22357#discussion_r3020549319
##########
components/camel-ai/camel-docling/src/test/java/org/apache/camel/component/docling/integration/DoclingServeProducerIT.java:
##########
@@ -245,14 +245,20 @@ void testCheckConversionStatus() throws Exception {
assertNotNull(taskId, "Task ID should not be null");
- // Wait a bit for processing
- Thread.sleep(1000);
+ // Poll until completed (the async future may take longer than a fixed sleep)
+ ConversionStatus status = null;
+ int maxAttempts = 120;
+ for (int i = 0; i < maxAttempts; i++) {
+ status = template.requestBody("direct:check-status", taskId, ConversionStatus.class);
+ assertNotNull(status, "Status should not be null");
+ assertNotNull(status.getTaskId(), "Status task ID should not be null");
- // Check status
- ConversionStatus status = template.requestBody("direct:check-status", taskId, ConversionStatus.class);
+ if (status.getStatus() == Status.COMPLETED || status.getStatus() == Status.FAILED) {
+ break;
+ }
+ Thread.sleep(1000);
Review Comment:
This needs to be addressed
--
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: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Bump ai.docling:docling-serve-client from 0.4.7 to 0.5.0 [camel]
Posted by "oscerd (via GitHub)" <gi...@apache.org>.
oscerd commented on PR #22357:
URL: https://github.com/apache/camel/pull/22357#issuecomment-4168153634
I wasn't updated on the compilation fixes. I'm going to tests everything locally
--
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: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Bump ai.docling:docling-serve-client from 0.4.7 to 0.5.0 [camel]
Posted by "oscerd (via GitHub)" <gi...@apache.org>.
oscerd commented on PR #22357:
URL: https://github.com/apache/camel/pull/22357#issuecomment-4168365179
```
[INFO] -------------------------------------------------------
[INFO] T E S T S
[INFO] -------------------------------------------------------
[INFO] Running org.apache.camel.component.docling.BatchProcessingTest
[INFO] Tests run: 8, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.635 s -- in org.apache.camel.component.docling.BatchProcessingTest
[INFO] Running org.apache.camel.component.docling.DoclingAsyncConversionTest
[INFO] Tests run: 5, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.249 s -- in org.apache.camel.component.docling.DoclingAsyncConversionTest
[INFO] Running org.apache.camel.component.docling.DoclingCliDeadlockTest
[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.021 s -- in org.apache.camel.component.docling.DoclingCliDeadlockTest
[INFO] Running org.apache.camel.component.docling.DoclingComponentTest
[INFO] Tests run: 5, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.060 s -- in org.apache.camel.component.docling.DoclingComponentTest
[INFO] Running org.apache.camel.component.docling.DoclingCustomArgsValidationTest
[INFO] Tests run: 15, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.550 s -- in org.apache.camel.component.docling.DoclingCustomArgsValidationTest
[INFO] Running org.apache.camel.component.docling.DoclingProducerTest
[WARNING] Tests run: 4, Failures: 0, Errors: 0, Skipped: 4, Time elapsed: 0.016 s -- in org.apache.camel.component.docling.DoclingProducerTest
[INFO] Running org.apache.camel.component.docling.DoclingSecureTempFileTest
[INFO] Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.030 s -- in org.apache.camel.component.docling.DoclingSecureTempFileTest
[INFO] Running org.apache.camel.component.docling.DoclingServeProducerTest
[WARNING] Tests run: 4, Failures: 0, Errors: 0, Skipped: 4, Time elapsed: 0.002 s -- in org.apache.camel.component.docling.DoclingServeProducerTest
[INFO] Running org.apache.camel.component.docling.DoclingTempFileCleanupTest
[INFO] Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.190 s -- in org.apache.camel.component.docling.DoclingTempFileCleanupTest
[INFO]
[INFO] Results:
[INFO]
[INFO] Tests run: 46, Failures: 0, Errors: 0, Skipped: 8
[INFO]
[INFO]
[INFO] --- camel-package:4.19.0-SNAPSHOT:generate-postcompile (generate-postcompile) @ camel-docling ---
[INFO] Skipping generation, everything is up to date.
[INFO]
[INFO] --- jar:3.5.0:jar (default-jar) @ camel-docling ---
[INFO] Building jar: /home/oscerd/workspace/apache-camel/camel/components/camel-ai/camel-docling/target/camel-docling-4.19.0-SNAPSHOT.jar
[INFO]
[INFO] --- failsafe:3.5.5:integration-test (default) @ camel-docling ---
[INFO] Using auto detected provider org.apache.maven.surefire.junitplatform.JUnitPlatformProvider
[INFO] Using auto detected provider org.apache.maven.surefire.junitplatform.JUnitPlatformProvider
[INFO] Using auto detected provider org.apache.maven.surefire.junitplatform.JUnitPlatformProvider
[INFO]
[INFO] -------------------------------------------------------
[INFO] T E S T S
[INFO] -------------------------------------------------------
[INFO] Running org.apache.camel.component.docling.integration.BatchProcessingIT
[INFO] Tests run: 8, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 29.07 s -- in org.apache.camel.component.docling.integration.BatchProcessingIT
[INFO] Running org.apache.camel.component.docling.integration.ChunkingIT
[INFO] Tests run: 6, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 26.89 s -- in org.apache.camel.component.docling.integration.ChunkingIT
[INFO] Running org.apache.camel.component.docling.integration.DoclingServeProducerIT
[INFO] Tests run: 16, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 108.8 s -- in org.apache.camel.component.docling.integration.DoclingServeProducerIT
[INFO] Running org.apache.camel.component.docling.integration.ExtractStructuredDataIT
[INFO] Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 84.70 s -- in org.apache.camel.component.docling.integration.ExtractStructuredDataIT
[INFO] Running org.apache.camel.component.docling.integration.MetadataExtractionIT
[INFO] Tests run: 10, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 43.77 s -- in org.apache.camel.component.docling.integration.MetadataExtractionIT
[INFO] Running org.apache.camel.component.docling.integration.OcrExtractionIT
[INFO] Tests run: 6, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 77.12 s -- in org.apache.camel.component.docling.integration.OcrExtractionIT
[INFO]
[INFO] Results:
[INFO]
[INFO] Tests run: 49, Failures: 0, Errors: 0, Skipped: 0
[INFO]
[INFO]
[INFO] --- failsafe:3.5.5:verify (default) @ camel-docling ---
```
--
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: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Bump ai.docling:docling-serve-client from 0.4.7 to 0.5.0 [camel]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #22357:
URL: https://github.com/apache/camel/pull/22357#issuecomment-4161683216
:star2: Thank you for your contribution to the Apache Camel project! :star2:
:robot: CI automation will test this PR automatically.
:camel: Apache Camel Committers, please review the following items:
* First-time contributors **require MANUAL approval** for the GitHub Actions to run
* You can use the command `/component-test (camel-)component-name1 (camel-)component-name2..` to request a test from the test bot although they are normally detected and executed by CI.
* You can label PRs using `build-all`, `build-dependents`, `skip-tests` and `test-dependents` to fine-tune the checks executed by this PR.
* Build and test logs are available in the summary page. **Only** [Apache Camel committers](https://camel.apache.org/community/team/#committers) have access to the summary.
:warning: Be careful when sharing logs. Review their contents before sharing them publicly.
--
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: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Bump ai.docling:docling-serve-client from 0.4.7 to 0.5.0 [camel]
Posted by "apupier (via GitHub)" <gi...@apache.org>.
apupier commented on PR #22357:
URL: https://github.com/apache/camel/pull/22357#issuecomment-4162923595
> Supersedes #22139 (Dependabot PR with compilation errors).
this PR didn't have compilation errors anymore as I provided fixes for the compilation side. It was remaining a failing test https://github.com/apache/camel/pull/22139#issuecomment-4098625285
--
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: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Bump ai.docling:docling-serve-client from 0.4.7 to 0.5.0 [camel]
Posted by "apupier (via GitHub)" <gi...@apache.org>.
apupier commented on PR #22357:
URL: https://github.com/apache/camel/pull/22357#issuecomment-4162915636
What's the difference with https://github.com/apache/camel/pull/22139/changes with which there was a failing test? i'm failing to see one.
--
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: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Bump ai.docling:docling-serve-client from 0.4.7 to 0.5.0 [camel]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #22357:
URL: https://github.com/apache/camel/pull/22357#issuecomment-4161797314
<!-- ci-tested-modules -->
:test_tube: **CI tested the following changed modules:**
- `components/camel-ai/camel-docling`
- `parent`
> :information_source: Dependent modules were not tested because the total number of affected modules exceeded the threshold (50). Use the `test-dependents` label to force testing all dependents.
<details><summary>Build reactor — dependencies compiled but only changed modules were tested (4 modules)</summary>
- Camel :: AI :: Docling
- Camel :: AI :: Docling [jar]
- Camel :: Parent
- Camel :: Parent [pom]
</details>
--
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: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Bump ai.docling:docling-serve-client from 0.4.7 to 0.5.0 [camel]
Posted by "oscerd (via GitHub)" <gi...@apache.org>.
oscerd merged PR #22357:
URL: https://github.com/apache/camel/pull/22357
--
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: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org