Skip to content

Add SyncService client from EGW for embedded calling from Data API#2405

Open
tatu-at-datastax wants to merge 19 commits intomainfrom
tatu/2387-add-egw-sync-service-client
Open

Add SyncService client from EGW for embedded calling from Data API#2405
tatu-at-datastax wants to merge 19 commits intomainfrom
tatu/2387-add-egw-sync-service-client

Conversation

@tatu-at-datastax
Copy link
Contributor

What this PR does:

Embeds sync-service validation call code from EGW into Data API, to prepare for removal of EGW

Which issue(s) this PR fixes:
Part of #2387

Checklist

  • Changes manually tested
  • Automated Tests added/updated
  • Documentation added/updated
  • CLA Signed: DataStax CLA

@tatu-at-datastax tatu-at-datastax self-assigned this Feb 21, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 21, 2026

📉 Unit Test Coverage Delta vs Main Branch

Metric Value
Main Branch 49.88%
This PR 49.56%
Delta 🔴 -0.32%
⚠️ Coverage decreased

@github-actions
Copy link
Contributor

github-actions bot commented Feb 21, 2026

Unit Test Coverage Report

Overall Project 49.56% -0.65% 🍏
Files changed 1.47%

File Coverage
DataVectorizer.java 50.98% 🍏
EmbeddingProviderFactory.java 20.49% -6.17%
SyncServiceClient.java 2.73% -97.27%
VectorizeConfigValidator.java 0% -1.7%
SyncServiceCredentialResolvingProvider.java 0%
SyncServiceErrorMessageMapper.java 0%

@github-actions
Copy link
Contributor

github-actions bot commented Feb 21, 2026

📉 Integration Test Coverage Delta vs Main Branch (dse69-it)

Metric Value
Main Branch 70.90%
This PR 70.51%
Delta 🔴 -0.40%
⚠️ Coverage decreased

@github-actions
Copy link
Contributor

github-actions bot commented Feb 21, 2026

Integration Test Coverage Report (dse69-it)

Overall Project 70.51% -0.58% 🍏
Files changed 11.34%

File Coverage
VectorizeConfigValidator.java 88.86% 🍏
EmbeddingProviderFactory.java 56.05% -3.21%
DataVectorizer.java 39.77% 🍏
SyncServiceClient.java 18.75% -81.25%
SyncServiceCredentialResolvingProvider.java 0%
SyncServiceErrorMessageMapper.java 0%

@github-actions
Copy link
Contributor

github-actions bot commented Feb 21, 2026

📉 Integration Test Coverage Delta vs Main Branch (hcd-it)

Metric Value
Main Branch 72.63%
This PR 72.23%
Delta 🔴 -0.41%
⚠️ Coverage decreased

@github-actions
Copy link
Contributor

github-actions bot commented Feb 21, 2026

Integration Test Coverage Report (hcd-it)

Overall Project 72.23% -0.58% 🍏
Files changed 11.34%

File Coverage
VectorizeConfigValidator.java 88.86% 🍏
EmbeddingProviderFactory.java 56.05% -3.21%
DataVectorizer.java 39.77% 🍏
SyncServiceClient.java 18.75% -81.25%
SyncServiceCredentialResolvingProvider.java 0%
SyncServiceErrorMessageMapper.java 0%

@tatu-at-datastax tatu-at-datastax changed the title (WIP) Add SyncService client from EGW Add SyncService client from EGW for embedded calling from Data API Mar 12, 2026
@tatu-at-datastax tatu-at-datastax marked this pull request as ready for review March 12, 2026 16:20
@tatu-at-datastax tatu-at-datastax requested a review from a team as a code owner March 12, 2026 16:20
@tatu-at-datastax
Copy link
Contributor Author

Ready for some review; not sure how to test.

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