Conversation
…ry refusals have their own types Two platform changes the reference had not caught up with. The Java SDK exposed only the raw body, so #problem-documents told Java readers to parse it themselves. DatahubApiException.problem() now returns a Problem (never null), and ingest follows the retry member rather than the status, honouring Retry-After. Platform branch sdk/problem-contract. Separately, datapoint-block-rejected no longer exists: each binary refusal carries the type its status calls for (invalid-frame, unknown-timeseries, request-too-large, value-type-mismatch, external-id-mismatch, too-many-in-flight, unsupported-media-type), with reason kept as the sub-code. That landed in platform b3412ec0 and was never documented. Rust and Python claims are unchanged: neither client changed. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Signed-off-by: jgjesdal <jostein@intellistream.ai>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two platform changes the reference had not caught up with. Both made existing pages wrong, which is worse than a missing page.
1. The Java SDK now parses the problem document
Pairs with IntelliStream-DataHub/datahub-platform#130.
client.md#problem-documentstold Java readers "there is no typed accessor for its members yet, so read it with your own JSON parser".DatahubApiException.problem()now returns aProblemthat is never null, so the Java tab gets the same treatment the Python and Rust tabs already had.Java's ingest retry also changed: it follows the problem's
retrymember rather than the status, and honoursRetry-After. That made two claims false:ingestretries429,5xxand network failures with backoff and surfaces everything else"Retry-After" — now true of Rust and Python only, so it is scoped to them.Rust and Python claims are untouched. Neither client changed, and neither was verified here.
2.
datapoint-block-rejectedno longer existsPlatform
b3412ec0(errors/9) gave each binary refusal the type its status calls for:invalid-frame,unknown-timeseries,request-too-large,value-type-mismatch,external-id-mismatch,too-many-in-flight,unsupported-media-type.reasonwas deliberately kept as the sub-code, so every table keyed byreasonstays correct; only thetypeclaims were wrong. That was never documented, and it left 5 stale references across 3 pages.Files
client.mdProblem;retry-is-advisory and the not-a-problem-document caution reconciled with Java's never-nullproblem(); retryable table row and the Java retry paragraph correctedlimits.md#sdk-behaviourtable and prose split Java from Rust/Pythonbinary-datapoints.md#responsesintro, JSON example and a newtypecolumn;#retryingnarrowed from5xxto502/503/504Verification
npm run buildcompiles clean (zero warnings, no broken links or anchors). No em-dashes added; zerodatapoint-block-rejectedleft indocs/.Found, not fixed
resources.md:1007breaksnpm run buildonmastertoday — the heading is indented two spaces, so MDX parses{#client-coverage}as a JS expression. Pre-existing, from the docs(java-sdk): the Java client now reaches every endpoint but two #94 merge. One-line fix (delete the indent), left out to keep this diff scoped.timeseries.md:557,:584,:770andguides/ingest-timeseries.mdx:154.AGENTS.mdsays system Node is 18 and documents a standalone-Node workaround; it is 22.22.2 on this host and the workaround was not needed.🤖 Generated with Claude Code