Skip to content

SK-3131 remove query interface - #430

Merged
skyflow-bharti merged 3 commits into
flowvault-release/26.9.1from
SK-3131-unary-operations-in-java-flowvault-sdk
Sep 10, 2026
Merged

SK-3131 remove query interface#430
skyflow-bharti merged 3 commits into
flowvault-release/26.9.1from
SK-3131-unary-operations-in-java-flowvault-sdk

Conversation

@skyflow-bharti

Copy link
Copy Markdown
Collaborator

No description provided.

@codecov

codecov Bot commented Sep 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.99%. Comparing base (e87efb3) to head (270d419).
⚠️ Report is 14 commits behind head on flowvault-release/26.9.1.

Additional details and impacted files
@@                      Coverage Diff                       @@
##             flowvault-release/26.9.1     #430      +/-   ##
==============================================================
+ Coverage                       90.74%   90.99%   +0.24%     
+ Complexity                        549      545       -4     
==============================================================
  Files                             179      174       -5     
  Lines                            7437     7350      -87     
  Branches                          998      989       -9     
==============================================================
- Hits                             6749     6688      -61     
+ Misses                            460      437      -23     
+ Partials                          228      225       -3     
Flag Coverage Δ
common 89.15% <ø> (ø)
flowvault 87.93% <ø> (+0.49%) ⬆️
skyvault 94.95% <ø> (ø)
unittests-flowvault 89.45% <ø> (+0.61%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
Module: common 89.15% <ø> (ø)
Module: skyvault 94.95% <ø> (ø)
Module: flowvault 87.93% <ø> (+0.49%) ⬆️
Service Account 87.36% <ø> (ø)
Vault Data 90.81% <ø> (+1.11%) ⬆️
Vault Tokens 99.03% <ø> (ø)
Vault Connection 100.00% <ø> (ø)
Vault Controller 82.75% <ø> (+0.41%) ⬆️
Detect 100.00% <ø> (ø)
Audit 100.00% <ø> (ø)
BIN Lookup 100.00% <ø> (ø)
Config 96.26% <ø> (ø)
Utils 90.36% <ø> (-0.08%) ⬇️
Errors 100.00% <ø> (ø)
Enums 100.00% <ø> (ø)
Logs 95.60% <ø> (ø)
Files with missing lines Coverage Δ
...owvault/src/main/java/com/skyflow/VaultClient.java 97.40% <ø> (-0.04%) ⬇️
...src/main/java/com/skyflow/enums/InterfaceName.java 100.00% <ø> (ø)
...owvault/src/main/java/com/skyflow/utils/Utils.java 91.78% <ø> (-0.03%) ⬇️
...ava/com/skyflow/utils/validations/Validations.java 90.70% <ø> (-0.27%) ⬇️
.../com/skyflow/vault/controller/VaultController.java 73.94% <ø> (+0.49%) ⬆️

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e87efb3...270d419. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions

github-actions Bot commented Sep 10, 2026

Copy link
Copy Markdown

Contract baseline change detected (flowvault)

This PR updates flowvault/api-report/skyflow-flowvault-java.baseline.jar (the approved public API contract). Here is exactly what it changes, comparing the baseline on flowvault-release/26.9.1 against the baseline committed in this PR:

Compatibility Report

semver MAJOR

Summary

Caution

Incompatible changes found while checking backward compatibility of version skyflow-flowvault-java.baseline with the previous version old-baseline.

Expand to see options used.
  • Report only summary: No
  • Report only changes: Yes
  • Report only binary-incompatible changes: No
  • Access modifier filter: PROTECTED
  • Old archives:
    • old-baseline unknown
  • New archives:
    • skyflow-flowvault-java.baseline unknown
  • Evaluate annotations: Yes
  • Include synthetic classes and class members: No
  • Include specific elements: Yes
    • com.skyflow.Skyflow
    • com.skyflow.config
    • com.skyflow.enums
    • com.skyflow.errors
    • com.skyflow.serviceaccount.util
    • com.skyflow.vault.audit
    • com.skyflow.vault.bin
    • com.skyflow.vault.connection
    • com.skyflow.vault.controller
    • com.skyflow.vault.data
    • com.skyflow.vault.detect
    • com.skyflow.vault.tokens
  • Exclude specific elements: No
  • Ignore all missing classes: Yes
  • Ignore specific missing classes: No
  • Treat changes as errors:
    • Any changes: No
    • Binary incompatible changes: No
    • Source incompatible changes: No
    • Incompatible changes caused by excluded classes: Yes
    • Semantically incompatible changes: No
    • Semantically incompatible changes, including development versions: No
  • Classpath mode: TWO_SEPARATE_CLASSPATHS
  • Old classpath:
/home/runner/.m2/repository/com/skyflow/common/1.0.0/common-1.0.0.jar:/home/runner/.m2/repository/com/fasterxml/jackson/core/jackson-databind/2.17.2/jackson-databind-2.17.2.jar:/home/runner/.m2/repository/com/fasterxml/jackson/core/jackson-annotations/2.17.2/jackson-annotations-2.17.2.jar:/home/runner/.m2/repository/com/fasterxml/jackson/core/jackson-core/2.17.2/jackson-core-2.17.2.jar:/home/runner/.m2/repository/com/fasterxml/jackson/datatype/jackson-datatype-jdk8/2.18.6/jackson-datatype-jdk8-2.18.6.jar:/home/runner/.m2/repository/com/fasterxml/jackson/datatype/jackson-datatype-jsr310/2.18.6/jackson-datatype-jsr310-2.18.6.jar:/home/runner/.m2/repository/io/github/cdimascio/dotenv-java/2.2.0/dotenv-java-2.2.0.jar:/home/runner/.m2/repository/com/google/code/gson/gson/2.10.1/gson-2.10.1.jar:/home/runner/.m2/repository/com/squareup/okhttp3/okhttp/4.12.0/okhttp-4.12.0.jar:/home/runner/.m2/repository/com/squareup/okio/okio/3.6.0/okio-3.6.0.jar:/home/runner/.m2/repository/com/squareup/okio/okio-jvm/3.6.0/okio-jvm-3.6.0.jar:/home/runner/.m2/repository/org/jetbrains/kotlin/kotlin-stdlib-common/1.9.10/kotlin-stdlib-common-1.9.10.jar:/home/runner/.m2/repository/org/jetbrains/kotlin/kotlin-stdlib-jdk8/1.8.21/kotlin-stdlib-jdk8-1.8.21.jar:/home/runner/.m2/repository/org/jetbrains/kotlin/kotlin-stdlib/1.8.21/kotlin-stdlib-1.8.21.jar:/home/runner/.m2/repository/org/jetbrains/annotations/13.0/annotations-13.0.jar:/home/runner/.m2/repository/org/jetbrains/kotlin/kotlin-stdlib-jdk7/1.8.21/kotlin-stdlib-jdk7-1.8.21.jar:/home/runner/.m2/repository/io/jsonwebtoken/jjwt/0.12.6/jjwt-0.12.6.jar:/home/runner/.m2/repository/io/jsonwebtoken/jjwt-api/0.12.6/jjwt-api-0.12.6.jar:/home/runner/.m2/repository/io/jsonwebtoken/jjwt-impl/0.12.6/jjwt-impl-0.12.6.jar:/home/runner/.m2/repository/io/jsonwebtoken/jjwt-jackson/0.12.6/jjwt-jackson-0.12.6.jar:/home/runner/.m2/repository/junit/junit/4.13.2/junit-4.13.2.jar:/home/runner/.m2/repository/org/hamcrest/hamcrest-core/1.3/hamcrest-core-1.3.jar:/home/runner/.m2/repository/org/powermock/powermock-module-junit4/2.0.9/powermock-module-junit4-2.0.9.jar:/home/runner/.m2/repository/org/powermock/powermock-module-junit4-common/2.0.9/powermock-module-junit4-common-2.0.9.jar:/home/runner/.m2/repository/org/powermock/powermock-reflect/2.0.9/powermock-reflect-2.0.9.jar:/home/runner/.m2/repository/net/bytebuddy/byte-buddy/1.10.14/byte-buddy-1.10.14.jar:/home/runner/.m2/repository/net/bytebuddy/byte-buddy-agent/1.10.14/byte-buddy-agent-1.10.14.jar:/home/runner/.m2/repository/org/powermock/powermock-core/2.0.9/powermock-core-2.0.9.jar:/home/runner/.m2/repository/org/javassist/javassist/3.27.0-GA/javassist-3.27.0-GA.jar:/home/runner/.m2/repository/org/powermock/powermock-api-mockito2/2.0.9/powermock-api-mockito2-2.0.9.jar:/home/runner/.m2/repository/org/powermock/powermock-api-support/2.0.9/powermock-api-support-2.0.9.jar:/home/runner/.m2/repository/org/mockito/mockito-core/3.3.3/mockito-core-3.3.3.jar:/home/runner/.m2/repository/org/objenesis/objenesis/2.6/objenesis-2.6.jar
  • New classpath:
/home/runner/.m2/repository/com/skyflow/common/1.0.0/common-1.0.0.jar:/home/runner/.m2/repository/com/fasterxml/jackson/core/jackson-databind/2.17.2/jackson-databind-2.17.2.jar:/home/runner/.m2/repository/com/fasterxml/jackson/core/jackson-annotations/2.17.2/jackson-annotations-2.17.2.jar:/home/runner/.m2/repository/com/fasterxml/jackson/core/jackson-core/2.17.2/jackson-core-2.17.2.jar:/home/runner/.m2/repository/com/fasterxml/jackson/datatype/jackson-datatype-jdk8/2.18.6/jackson-datatype-jdk8-2.18.6.jar:/home/runner/.m2/repository/com/fasterxml/jackson/datatype/jackson-datatype-jsr310/2.18.6/jackson-datatype-jsr310-2.18.6.jar:/home/runner/.m2/repository/io/github/cdimascio/dotenv-java/2.2.0/dotenv-java-2.2.0.jar:/home/runner/.m2/repository/com/google/code/gson/gson/2.10.1/gson-2.10.1.jar:/home/runner/.m2/repository/com/squareup/okhttp3/okhttp/4.12.0/okhttp-4.12.0.jar:/home/runner/.m2/repository/com/squareup/okio/okio/3.6.0/okio-3.6.0.jar:/home/runner/.m2/repository/com/squareup/okio/okio-jvm/3.6.0/okio-jvm-3.6.0.jar:/home/runner/.m2/repository/org/jetbrains/kotlin/kotlin-stdlib-common/1.9.10/kotlin-stdlib-common-1.9.10.jar:/home/runner/.m2/repository/org/jetbrains/kotlin/kotlin-stdlib-jdk8/1.8.21/kotlin-stdlib-jdk8-1.8.21.jar:/home/runner/.m2/repository/org/jetbrains/kotlin/kotlin-stdlib/1.8.21/kotlin-stdlib-1.8.21.jar:/home/runner/.m2/repository/org/jetbrains/annotations/13.0/annotations-13.0.jar:/home/runner/.m2/repository/org/jetbrains/kotlin/kotlin-stdlib-jdk7/1.8.21/kotlin-stdlib-jdk7-1.8.21.jar:/home/runner/.m2/repository/io/jsonwebtoken/jjwt/0.12.6/jjwt-0.12.6.jar:/home/runner/.m2/repository/io/jsonwebtoken/jjwt-api/0.12.6/jjwt-api-0.12.6.jar:/home/runner/.m2/repository/io/jsonwebtoken/jjwt-impl/0.12.6/jjwt-impl-0.12.6.jar:/home/runner/.m2/repository/io/jsonwebtoken/jjwt-jackson/0.12.6/jjwt-jackson-0.12.6.jar:/home/runner/.m2/repository/junit/junit/4.13.2/junit-4.13.2.jar:/home/runner/.m2/repository/org/hamcrest/hamcrest-core/1.3/hamcrest-core-1.3.jar:/home/runner/.m2/repository/org/powermock/powermock-module-junit4/2.0.9/powermock-module-junit4-2.0.9.jar:/home/runner/.m2/repository/org/powermock/powermock-module-junit4-common/2.0.9/powermock-module-junit4-common-2.0.9.jar:/home/runner/.m2/repository/org/powermock/powermock-reflect/2.0.9/powermock-reflect-2.0.9.jar:/home/runner/.m2/repository/net/bytebuddy/byte-buddy/1.10.14/byte-buddy-1.10.14.jar:/home/runner/.m2/repository/net/bytebuddy/byte-buddy-agent/1.10.14/byte-buddy-agent-1.10.14.jar:/home/runner/.m2/repository/org/powermock/powermock-core/2.0.9/powermock-core-2.0.9.jar:/home/runner/.m2/repository/org/javassist/javassist/3.27.0-GA/javassist-3.27.0-GA.jar:/home/runner/.m2/repository/org/powermock/powermock-api-mockito2/2.0.9/powermock-api-mockito2-2.0.9.jar:/home/runner/.m2/repository/org/powermock/powermock-api-support/2.0.9/powermock-api-support-2.0.9.jar:/home/runner/.m2/repository/org/mockito/mockito-core/3.3.3/mockito-core-3.3.3.jar:/home/runner/.m2/repository/org/objenesis/objenesis/2.6/objenesis-2.6.jar

Results

Status Type Serialization Compatibility Changes
Modified com.skyflow.enums.InterfaceName Field removed Field removed
Modified com.skyflow.vault.controller.VaultController Not serializable Method removed
Removed com.skyflow.vault.data.QueryOptions Not serializable Class removed Superclass removed Method removed Constructor removed
Removed com.skyflow.vault.data.QueryOptions$Builder Not serializable Class removed Superclass removed Method removed Constructor removed
Removed com.skyflow.vault.data.QueryRequest Not serializable Class removed Superclass removed Method removed Constructor removed
Removed com.skyflow.vault.data.QueryRequest$QueryRequestBuilder Not serializable Class removed Superclass removed Method removed
Removed com.skyflow.vault.data.QueryResponse Not serializable Class removed Superclass removed Method removed Constructor removed
Removed com.skyflow.vault.data.QueryResponseMetadata Not serializable Class removed Superclass removed Method removed Constructor removed
Removed com.skyflow.vault.data.QueryResponseRecord Not serializable Class removed Superclass removed Method removed Constructor removed
Expand for details.

com.skyflow.enums.InterfaceName

  • Binary-compatible
  • Source-compatible
  • Serialization-compatible
Status Modifiers Type Name Extends JDK Serialization Compatibility Changes
Modified final public Enum InterfaceName Enum<E> JDK 8 Field removed No changes

Fields

Status Modifiers Type Name Annotations Compatibility Changes
Removed public static final InterfaceName QUERY Field removed

com.skyflow.vault.controller.VaultController

  • Binary-compatible
  • Source-compatible
  • Serialization-compatible
Status Modifiers Type Name Extends JDK Serialization Compatibility Changes
Modified final public Class VaultController null JDK 8 Not serializable No changes

Methods

Status Modifiers Generics Type Method Annotations Throws Compatibility Changes
Removed public QueryResponse query(QueryRequest) SkyflowException Method removed
Removed public QueryResponse query(QueryRequest, QueryOptions) SkyflowException Method removed

com.skyflow.vault.data.QueryOptions

  • Binary-compatible
  • Source-compatible
  • Serialization-compatible
Status Modifiers Type Name Extends JDK Serialization Compatibility Changes
Removed public Class QueryOptions Object JDK 8 Not serializable Class removed Superclass removed

Constructors

Status Modifiers Generics Constructor Annotations Throws Compatibility Changes
Removed protected QueryOptions(Builder) Constructor removed

Methods

Status Modifiers Generics Type Method Annotations Throws Compatibility Changes
Removed static public Builder builder() Method removed
Removed public RequestInterceptor getInterceptor() Method removed

com.skyflow.vault.data.QueryOptions$Builder

  • Binary-compatible
  • Source-compatible
  • Serialization-compatible
Status Modifiers Type Name Extends JDK Serialization Compatibility Changes
Removed static public Class Builder Object JDK 8 Not serializable Class removed Superclass removed

Constructors

Status Modifiers Generics Constructor Annotations Throws Compatibility Changes
Removed protected Builder() Constructor removed

Methods

Status Modifiers Generics Type Method Annotations Throws Compatibility Changes
Removed public QueryOptions build() Method removed
Removed public Builder interceptor(RequestInterceptor) Method removed

com.skyflow.vault.data.QueryRequest

  • Binary-compatible
  • Source-compatible
  • Serialization-compatible
Status Modifiers Type Name Extends JDK Serialization Compatibility Changes
Removed public Class QueryRequest BaseQueryRequest JDK 8 Not serializable Class removed Superclass removed

Constructors

Status Modifiers Generics Constructor Annotations Throws Compatibility Changes
Removed protected QueryRequest(QueryRequestBuilder) Constructor removed

Methods

Status Modifiers Generics Type Method Annotations Throws Compatibility Changes
Removed static public QueryRequestBuilder builder() Method removed

com.skyflow.vault.data.QueryRequest$QueryRequestBuilder

  • Binary-compatible
  • Source-compatible
  • Serialization-compatible
Status Modifiers Type Name Extends JDK Serialization Compatibility Changes
Removed final static public Class QueryRequestBuilder BaseQueryRequestBuilder JDK 8 Not serializable Class removed Superclass removed

Methods

Status Modifiers Generics Type Method Annotations Throws Compatibility Changes
Removed public QueryRequest build() Method removed
Removed public QueryRequestBuilder query(String) Method removed

com.skyflow.vault.data.QueryResponse

  • Binary-compatible
  • Source-compatible
  • Serialization-compatible
Status Modifiers Type Name Extends JDK Serialization Compatibility Changes
Removed public Class QueryResponse Object JDK 8 Not serializable Class removed Superclass removed

Constructors

Status Modifiers Generics Constructor Annotations Throws Compatibility Changes
Removed public QueryResponse(List<QueryResponseRecord>, QueryResponseMetadata) Constructor removed

Methods

Status Modifiers Generics Type Method Annotations Throws Compatibility Changes
Removed public QueryResponseMetadata getMetadata() Method removed
Removed public List<QueryResponseRecord> getRecords() Method removed
Removed public String toString() Method removed

com.skyflow.vault.data.QueryResponseMetadata

  • Binary-compatible
  • Source-compatible
  • Serialization-compatible
Status Modifiers Type Name Extends JDK Serialization Compatibility Changes
Removed public Class QueryResponseMetadata Object JDK 8 Not serializable Class removed Superclass removed

Constructors

Status Modifiers Generics Constructor Annotations Throws Compatibility Changes
Removed public QueryResponseMetadata(List<String>) Constructor removed

Methods

Status Modifiers Generics Type Method Annotations Throws Compatibility Changes
Removed public List<String> getColumns() Method removed
Removed public String toString() Method removed

com.skyflow.vault.data.QueryResponseRecord

  • Binary-compatible
  • Source-compatible
  • Serialization-compatible
Status Modifiers Type Name Extends JDK Serialization Compatibility Changes
Removed public Class QueryResponseRecord Object JDK 8 Not serializable Class removed Superclass removed

Constructors

Status Modifiers Generics Constructor Annotations Throws Compatibility Changes
Removed public QueryResponseRecord(Map<String, Object>) Constructor removed

Methods

Status Modifiers Generics Type Method Annotations Throws Compatibility Changes
Removed public Map<String, Object> getData() Method removed
Removed public String toString() Method removed

Warning

All missing classes, i.e. superclasses and interfaces that could not be found on the classpath were ignored.

Hence changes caused by these superclasses and interfaces are not reflected in the output.


Generated on: 2026-09-10 07:16:45.872+0000.

@skyflow-bharti
skyflow-bharti merged commit 28bad97 into flowvault-release/26.9.1 Sep 10, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant