[bot] Google GenAI Batches API is not instrumented (mis-tagged as a generic LLM span)
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 38/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Tranquilla
- Stack tecnologico
- java
- Ambito
- observability-sre
Direzione di ricerca
Inizia da braintrust-sdk/instrumentation/genai_1_18_0/src/main/java/com/google/genai/BraintrustApiClient.java, in particolare da tagSpan(), quindi esamina BraintrustInstrumentation.java per confermare il client e i punti di ingresso asincroni dei batch. Confronta le strutture della google-genai Batches API con l’instrumentation esistente e individua i test da aggiungere per create, get, list, cancel e i risultati completati. Il lavoro è completo quando le operazioni batch dispongono di dati span appropriati e di copertura di regressione, senza trattare i metadati di BatchJob come output della generazione.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Summary
The genai_1_18_0 module instruments Google's google-genai Java SDK by subclassing its package-private ApiClient (com.google.genai.BraintrustApiClient) and swapping it into every service object on the Client, including client.batches and client.async.batches (BraintrustInstrumentation.wrapClient, lines 36–52). This means a call to client.batches.create(...)/.get(...)/.list(...)/.cancel(...)/.createEmbeddings(...) does produce a span — but BraintrustApiClient.tagSpan() has no awareness of the Batches API's request/response shape, so the span is mis-tagged: it's marked span_attributes.type = "llm" as if it were a real generation call, but the request/response field extraction (built for generateContent-shaped bodies) finds essentially nothing meaningful to populate.
What is missing
In braintrust-sdk/instrumentation/genai_1_18_0/src/main/java/com/google/genai/BraintrustApiClient.java, tagSpan() (lines 50–161):
- Request metadata extraction (lines 65–76) looks for top-level
model,systemInstruction,tools,toolConfig,safetySettings,cachedContent. AbatchGenerateContentrequest body wraps everything under abatchobject ({"batch": {"displayName": ..., "inputConfig": {...}}}), and acreateEmbeddingsbatch job body is shaped aroundEmbeddingsBatchJobSource/CreateEmbeddingsBatchJobConfig— none of the expected top-level fields exist, so metadata stays essentially empty. input_jsonconstruction (lines 97–113) only populatesmodel/contents/config(fromgenerationConfig) — a batch-create body has none of these at top level (the model is only present in the URL path, e.g.{model}:batchGenerateContent, andgetModel(genAIEndpoint)(used as a fallback at line 102) is the only path by whichmodelwould end up ininput_jsonat all); the batch's actual per-itemcontents/generation requests (whether inline or file/GCS-referenced) are never captured.- Response handling (lines 116–150) dumps the whole response body as
output_json(line 125) — for a batch create/get call, that whole body is justBatchJobresource metadata (name,state,createTime, etc.), not generative output — and readsusageMetadatafor metrics (line 128), which aBatchJobresponse never has. There is no instrumentation at all of retrieving a completed batch's actual per-request results (where real generation outputs andusageMetadatabecome available), so even a fully successful batch job produces a span withtype: "llm"but no meaningful input, output, or token metrics. - No test or example anywhere in the repo exercises
client.batchesin any form (confirmed via grep forbatchunderbraintrust-sdk/instrumentation/genai_1_18_0/— the only match isBraintrustInstrumentation.java's own field-swap wiring, not a test).
Braintrust docs status: not_found
Checked https://www.braintrust.dev/docs/integrations/ai-providers/gemini in full: no mention of batchGenerateContent, Batches, or batch/async generation jobs anywhere, in any language. The only "batch"-adjacent mention is batchEmbedContents, and only in the unrelated context of AI proxy/gateway passthrough support for embeddings ("The gateway also supports Gemini's native embedContent and batchEmbedContents endpoints") — not span/tracing coverage, and not the Batches job API this issue is about.
Note: this repo's own gap-audit history already treats "Batch API not instrumented, mis-tagged as a generic LLM span" as a valid, in-scope finding — see the already-filed and still-open #155 for Anthropic's Message Batches API, which this issue mirrors for the Google GenAI provider (a distinct upstream API/SDK/module, not a duplicate).
Upstream sources
- Gemini API "Batch Mode" (
batchGenerateContent) — official docs: https://ai.google.dev/gemini-api/docs/batch-api and https://ai.google.dev/gemini-api/docs/batch-mode. Async, 50%-discounted, ~24h turnaround; supports inline requests or file/JSONL upload. - Vertex AI batch predictions for Gemini (and partner models) via GCS/BigQuery: https://docs.cloud.google.com/vertex-ai/generative-ai/docs/maas/capabilities/batch-prediction, with a Java-specific sample using the same
com.google.genai.Batchesclass: https://docs.cloud.google.com/vertex-ai/generative-ai/docs/samples/googlegenaisdk-batchpredict-with-gcs. - Official
google-genaiJava SDK exposes this directly:com.google.genai.Batches(googleapis/java-genai,src/main/java/com/google/genai/Batches.java), javadoc at https://googleapis.github.io/java-genai/javadoc/com/google/genai/Batches.html —create(String model, BatchJobSource, CreateBatchJobConfig),createEmbeddings(...)(experimental),get(...),cancel(...),delete(...),list(...), all returning/operating onBatchJob.
Local repo files inspected
braintrust-sdk/instrumentation/genai_1_18_0/src/main/java/com/google/genai/BraintrustApiClient.java— lines 50–161 (tagSpan)braintrust-sdk/instrumentation/genai_1_18_0/src/main/java/com/google/genai/BraintrustInstrumentation.java— lines 22–57 (wrapClient), confirmingclient.batches/client.async.batchesare explicitly wired to the same instrumentedApiClient(lines 37, 47) and therefore do produce (mis-tagged) spans rather than bypassing instrumentation entirely- Repo-wide grep for
batch/Batch/batchGenerateContent/BatchJobunderbraintrust-sdk/instrumentation/genai_1_18_0/— only match is the field-swap wiring above; no test or example exercises the Batches API
- Lingua principale
- Java
- Stelle
- 21
- Fork
- 5
- Merge medio
- 2g 7h
- PR unite (30g)
- 8
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di braintrustdata/braintrust-sdk-java
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
Tutte le issue di braintrustdata/braintrust-sdk-java
Issue simili
-
executions.Query — startDate and timeRange filters are sent with inverted comparison operators Apertaarea/plugin
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
kestra-io/plugin-kestra#190 ·
-
litertlm-android AAR ships no consumer ProGuard rules → "mid == null" SIGABRT in minified apps Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
google-ai-edge/LiteRT-LM#3739 ·
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
integra-team-red/meet-map#249 ·
-
[Studio][Bug] Cancelled create-user dialog keeps the password and admin switch for the next attempt Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
apache/rocketmq-dashboard#5064 ·