Skip to content

feat(sdk): DSPX-4584 stream the TDF manifest, remove the ~21.7M segment ceiling - #399

Open
sujankota wants to merge 2 commits into
mainfrom
feat/dspx-4584-streaming-manifest
Open

feat(sdk): DSPX-4584 stream the TDF manifest, remove the ~21.7M segment ceiling#399
sujankota wants to merge 2 commits into
mainfrom
feat/dspx-4584-streaming-manifest

fix(sdk): bound segment hash stride, keep appendManifest as deprecated

4d08412
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Sep 11, 2026 in 41s

Annotations

Check warning on line 149 in sdk/src/main/java/io/opentdf/platform/sdk/Manifest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Override the "equals" method in this class.

See more on https://sonarcloud.io/project/issues?id=opentdf_java-sdk&issues=AaCHERClZxszJedvSZNh&open=AaCHERClZxszJedvSZNh&pullRequest=399

Check warning on line 161 in sdk/src/test/java/io/opentdf/platform/sdk/SegmentsTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use assertThat(actual).hasSize(expected) instead.

See more on https://sonarcloud.io/project/issues?id=opentdf_java-sdk&issues=AaCHEREsZxszJedvSZNo&open=AaCHEREsZxszJedvSZNo&pullRequest=399

Check warning on line 104 in sdk/src/test/java/io/opentdf/platform/sdk/SegmentsTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Join these multiple assertions subject to one assertion chain.

See more on https://sonarcloud.io/project/issues?id=opentdf_java-sdk&issues=AaCHEREsZxszJedvSZNn&open=AaCHEREsZxszJedvSZNn&pullRequest=399

Check warning on line 33 in sdk/src/main/java/io/opentdf/platform/sdk/TDFWriter.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Do not forget to remove this deprecated code someday.

See more on https://sonarcloud.io/project/issues?id=opentdf_java-sdk&issues=AaCRi9DeZT4GOkgS9dK9&open=AaCRi9DeZT4GOkgS9dK9&pullRequest=399

Check warning on line 118 in sdk/src/test/java/io/opentdf/platform/sdk/SegmentsTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Join these multiple assertions subject to one assertion chain.

See more on https://sonarcloud.io/project/issues?id=opentdf_java-sdk&issues=AaCRi9IFZT4GOkgS9dK-&open=AaCRi9IFZT4GOkgS9dK-&pullRequest=399

Check warning on line 32 in sdk/src/main/java/io/opentdf/platform/sdk/TDFWriter.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add 'since' and/or 'forRemoval' arguments to the @Deprecated annotation.

See more on https://sonarcloud.io/project/issues?id=opentdf_java-sdk&issues=AaCRi9DeZT4GOkgS9dK8&open=AaCRi9DeZT4GOkgS9dK8&pullRequest=399

Check warning on line 76 in sdk/src/test/java/io/opentdf/platform/sdk/SegmentsTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Join these multiple assertions subject to one assertion chain.

See more on https://sonarcloud.io/project/issues?id=opentdf_java-sdk&issues=AaCHEREsZxszJedvSZNl&open=AaCHEREsZxszJedvSZNl&pullRequest=399

Check failure on line 220 in sdk/src/test/java/io/opentdf/platform/sdk/SegmentsTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

See more on https://sonarcloud.io/project/issues?id=opentdf_java-sdk&issues=AaCHEREsZxszJedvSZNq&open=AaCHEREsZxszJedvSZNq&pullRequest=399

Check failure on line 224 in sdk/src/test/java/io/opentdf/platform/sdk/SegmentsTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

See more on https://sonarcloud.io/project/issues?id=opentdf_java-sdk&issues=AaCHEREsZxszJedvSZNr&open=AaCHEREsZxszJedvSZNr&pullRequest=399

Check warning on line 304 in sdk/src/main/java/io/opentdf/platform/sdk/Manifest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Return an empty collection instead of null.

See more on https://sonarcloud.io/project/issues?id=opentdf_java-sdk&issues=AaCHERClZxszJedvSZNi&open=AaCHERClZxszJedvSZNi&pullRequest=399

Check warning on line 91 in sdk/src/test/java/io/opentdf/platform/sdk/SegmentsTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Join these multiple assertions subject to one assertion chain.

See more on https://sonarcloud.io/project/issues?id=opentdf_java-sdk&issues=AaCHEREsZxszJedvSZNm&open=AaCHEREsZxszJedvSZNm&pullRequest=399

Check failure on line 207 in sdk/src/test/java/io/opentdf/platform/sdk/SegmentsTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

See more on https://sonarcloud.io/project/issues?id=opentdf_java-sdk&issues=AaCHEREsZxszJedvSZNp&open=AaCHEREsZxszJedvSZNp&pullRequest=399

Check warning on line 23 in sdk/src/test/java/io/opentdf/platform/sdk/SegmentsTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this 'public' modifier.

See more on https://sonarcloud.io/project/issues?id=opentdf_java-sdk&issues=AaCHEREsZxszJedvSZNk&open=AaCHEREsZxszJedvSZNk&pullRequest=399

Check failure on line 775 in sdk/src/main/java/io/opentdf/platform/sdk/Manifest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 20 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=opentdf_java-sdk&issues=AaCHERClZxszJedvSZNj&open=AaCHERClZxszJedvSZNj&pullRequest=399