feat: Support APIs for chunking sources #146

ThomasVitale · 2025-12-03T22:37:07Z

Add APIs for the /v1/chunk/hierarchical/source endpoint.
Add APIs for the /v1/chunk/hybrid/source endpoint.
Implement new endpoints in reference client.

github-actions · 2025-12-03T22:46:50Z

	Tests	Passed ✅	Skipped	Failed
Gradle Test Results (all modules & JDKs)	528 ran	528 passed	0 skipped	0 failed

Test	Result
No test annotations available

View Gradle Test Results (all modules & JDKs)

github-actions · 2025-12-03T22:46:51Z

HTML test reports are available as workflow artifacts (zipped HTML).

• Download: Artifacts for this run

* Add APIs for the /v1/chunk/hierarchical/source endpoint. * Add APIs for the /v1/chunk/hybrid/source endpoint. * Implement new endpoints in reference client. Fixes gh-144 Signed-off-by: Thomas Vitale <ThomasVitale@users.noreply.github.com>

github-actions · 2025-12-04T07:33:55Z

HTML test reports are available as workflow artifacts (zipped HTML).

• Download: Artifacts for this run

Signed-off-by: Eric Deandrea <eric.deandrea@ibm.com>

github-actions · 2025-12-04T14:28:30Z

HTML test reports are available as workflow artifacts (zipped HTML).

• Download: Artifacts for this run

edeandrea

Looks good!

I made one small refactoring in DoclingServeClient so we don't have to duplicate/copy-paste the creation of the HttpRequest. Let me know what you think.

Also, what about adding some of these tests to the version tester utility that I built which tests various docling-serve versions?

If these features were introduced in some docling serve version, we can do that too.

See

docling-java/docling-testing/docling-version-tests/src/main/java/ai/docling/client/tester/service/TagsTester.java

Lines 50 to 90 in 676a8ed

    
           private TagTestResult testTag(TagsTestRequest request, String tag) { 
        
             var registry = request.registry(); 
        
             var image = request.image(); 
        
             Log.infof("Testing %s/%s:%s", registry, image, tag); 
        
             var resultBuilder = TagTestResult.builder().tag(tag); 
        
             var containerConfig = DoclingServeContainerConfig.builder() 
        
                 .image("%s/%s:%s".formatted(registry, image, tag)) 
        
                 .startupTimeout(Duration.ofMinutes(5)) 
        
                 .build(); 
        
             try (var doclingContainer = new DoclingServeContainer(containerConfig)) { 
        
               try { 
        
                 doclingContainer.start(); 
        
                 doConversion(doclingContainer); 
        
                 resultBuilder.result(Result.success("Tag %s is ok".formatted(tag))); 
        
               } 
        
               finally { 
        
                 resultBuilder.serverLogs(doclingContainer.getLogs()); 
        
               } 
        
             } 
        
             catch (AssertionError | Exception ex) { 
        
               resultBuilder.result(Result.failure(ex)); 
        
             } 
        
             finally { 
        
               if (request.cleanupContainerImages()) { 
        
                 // Clean up the image to save on disk space 
        
                 DockerClientFactory.instance() 
        
                     .client() 
        
                     .removeImageCmd(containerConfig.image()) 
        
                     .withForce(true) 
        
                     .exec(); 
        
               } 
        
             } 
        
             var result = resultBuilder.build(); 
        
             Log.infof("Result [%s] = %s", result.tag(), result.result().status()); 
        
             return result; 
        
           }

ThomasVitale · 2025-12-04T14:34:25Z

Looks good, thanks! Is it ok if I look into the version tests in a separate PR?

Signed-off-by: Eric Deandrea <eric.deandrea@ibm.com>

github-actions · 2025-12-04T14:37:07Z

HTML test reports are available as workflow artifacts (zipped HTML).

• Download: Artifacts for this run

edeandrea · 2025-12-04T14:38:28Z

Is it ok if I look into the version tests in a separate PR?

Sure thing! I have #134 open, so that might help close out that issue.

Signed-off-by: Eric Deandrea <eric.deandrea@ibm.com>

github-actions · 2025-12-04T14:47:19Z

HTML test reports are available as workflow artifacts (zipped HTML).

• Download: Artifacts for this run

github-actions · 2025-12-04T14:55:12Z

HTML test reports are available as workflow artifacts (zipped HTML).

• Download: Artifacts for this run

docling-java-ops · 2025-12-08T18:18:05Z

🎉 This issue has been resolved in v0.2.0 (Release Notes)

ThomasVitale requested a review from edeandrea December 3, 2025 22:37

feat: Support APIs for chunking sources

52a06c1

* Add APIs for the /v1/chunk/hierarchical/source endpoint. * Add APIs for the /v1/chunk/hybrid/source endpoint. * Implement new endpoints in reference client. Fixes gh-144 Signed-off-by: Thomas Vitale <ThomasVitale@users.noreply.github.com>

ThomasVitale force-pushed the gh-144 branch from ca9b679 to 52a06c1 Compare December 4, 2025 07:20

feat: Small refactoring

a04378c

Signed-off-by: Eric Deandrea <eric.deandrea@ibm.com>

edeandrea force-pushed the gh-144 branch from fa6a4dc to a04378c Compare December 4, 2025 14:24

edeandrea previously approved these changes Dec 4, 2025

View reviewed changes

feat: Small refactoring

d361d33

Signed-off-by: Eric Deandrea <eric.deandrea@ibm.com>

edeandrea dismissed their stale review via d361d33 December 4, 2025 14:36

feat: Small refactoring

8dc58a9

Signed-off-by: Eric Deandrea <eric.deandrea@ibm.com>

edeandrea enabled auto-merge (squash) December 4, 2025 14:43

edeandrea merged commit f99c6b8 into main Dec 4, 2025
20 checks passed

edeandrea deleted the gh-144 branch December 4, 2025 14:53

docling-java-ops bot added the released Issue has been released label Dec 8, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat: Support APIs for chunking sources #146

feat: Support APIs for chunking sources #146

Uh oh!

ThomasVitale commented Dec 3, 2025

Uh oh!

github-actions bot commented Dec 3, 2025 •

edited

Loading

Uh oh!

github-actions bot commented Dec 3, 2025

Uh oh!

github-actions bot commented Dec 4, 2025

Uh oh!

github-actions bot commented Dec 4, 2025

Uh oh!

edeandrea left a comment

Uh oh!

ThomasVitale commented Dec 4, 2025

Uh oh!

github-actions bot commented Dec 4, 2025

Uh oh!

edeandrea commented Dec 4, 2025

Uh oh!

github-actions bot commented Dec 4, 2025

Uh oh!

Uh oh!

github-actions bot commented Dec 4, 2025

Uh oh!

docling-java-ops bot commented Dec 8, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

	private TagTestResult testTag(TagsTestRequest request, String tag) {
	var registry = request.registry();
	var image = request.image();

	Log.infof("Testing %s/%s:%s", registry, image, tag);
	var resultBuilder = TagTestResult.builder().tag(tag);
	var containerConfig = DoclingServeContainerConfig.builder()
	.image("%s/%s:%s".formatted(registry, image, tag))
	.startupTimeout(Duration.ofMinutes(5))
	.build();

	try (var doclingContainer = new DoclingServeContainer(containerConfig)) {
	try {
	doclingContainer.start();
	doConversion(doclingContainer);
	resultBuilder.result(Result.success("Tag %s is ok".formatted(tag)));
	}
	finally {
	resultBuilder.serverLogs(doclingContainer.getLogs());
	}
	}
	catch (AssertionError \| Exception ex) {
	resultBuilder.result(Result.failure(ex));
	}
	finally {
	if (request.cleanupContainerImages()) {
	// Clean up the image to save on disk space
	DockerClientFactory.instance()
	.client()
	.removeImageCmd(containerConfig.image())
	.withForce(true)
	.exec();
	}
	}

	var result = resultBuilder.build();

	Log.infof("Result [%s] = %s", result.tag(), result.result().status());

	return result;
	}

feat: Support APIs for chunking sources #146

feat: Support APIs for chunking sources #146

Uh oh!

Conversation

ThomasVitale commented Dec 3, 2025

Uh oh!

github-actions bot commented Dec 3, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions bot commented Dec 3, 2025

Uh oh!

github-actions bot commented Dec 4, 2025

Uh oh!

github-actions bot commented Dec 4, 2025

Uh oh!

edeandrea left a comment

Choose a reason for hiding this comment

Uh oh!

ThomasVitale commented Dec 4, 2025

Uh oh!

github-actions bot commented Dec 4, 2025

Uh oh!

edeandrea commented Dec 4, 2025

Uh oh!

github-actions bot commented Dec 4, 2025

Uh oh!

Uh oh!

github-actions bot commented Dec 4, 2025

Uh oh!

docling-java-ops bot commented Dec 8, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

github-actions bot commented Dec 3, 2025 •

edited

Loading