feat: [Batch]#850
Conversation
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
Jonas-Isr
left a comment
Jonas-Isr
left a comment
There was a problem hiding this comment.
Looks good, this is more than a PoC. Looks like it could be released.
@jjtang1985 What do you think regarding releasing this?
| RESOURCE_GROUP, | ||
| BatchCreateRequest.create() | ||
| .type(TypeEnum.LLM_NATIVE) | ||
| .input(BatchCreateRequestInput.create().uri("ai://s3secret/input-batch.jsonl")) |
There was a problem hiding this comment.
FYI, I checked, even if we use PUT request of the File API to create a file on the fly, it still needs the s3 bucket to upload.
I was thinking to not have test data configured independently from the test so that tests do not rely on any data from outside.
I would create this file on the fly (fine if we override). But no strong opinion.
| public static final String RESOURCE_GROUP = "ai-sdk-java-e2e"; | ||
|
|
||
| /** Directory path for batch output files in the object store. */ | ||
| public static final String S3_DIRECTORY = "s3secret//"; |
There was a problem hiding this comment.
Unfortunately, I think the service team decide to put a / in front of the actual s3 file name if the output uri is ai://s3secret/.
2026-04-28 15:31:23 2582 /e3eb3a73-8072-404e-bc00-123412341234/output.jsonl
2026-04-15 16:43:39 407 file.txtSo /e3eb3a73-8072-404e-bc00-123412341234/output.jsonl should never have / at the beginning.
But if you change the output to ai://s3secret/output, it is actually correct:
2026-05-07 14:08:46 4002 output/fa173027-a1ed-4b70-a147-200fa159a98d/output.jsonlThis is IMO a bug from their side.
There was a problem hiding this comment.
I changed the output folder
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> Co-authored-by: Jonas-Isr <jonas.israel@sap.com>
Co-authored-by: Jonas-Isr <jonas.israel@sap.com>
5 participants
Context
AI/ai-sdk-java-backlog#385.
New module, new API
Definition of Done