Skip to content

Conversation

@Hazel-Datastax
Copy link
Contributor

@Hazel-Datastax Hazel-Datastax commented Sep 29, 2025

What this PR does:
Replace stargate.jsonapi to data-api

Which issue(s) this PR fixes:
Fixes #2209

Checklist

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

@Hazel-Datastax Hazel-Datastax requested a review from a team as a code owner September 29, 2025 18:03
@Hazel-Datastax Hazel-Datastax marked this pull request as draft September 29, 2025 18:03
@Hazel-Datastax Hazel-Datastax marked this pull request as ready for review September 29, 2025 19:49
@tatu-at-datastax
Copy link
Contributor

I think there is also need to fix "env" versions of these settings:

  • docker-compose/docker-compose.yml and docker-compose/docker-compose-hcd.yml have entries like:
    • - STARGATE_JSONAPI_OPERATIONS_DATABASE_CONFIG_CASSANDRA_END_POINTS=hcd

@Hazel-Datastax
Copy link
Contributor Author

Hazel-Datastax commented Sep 30, 2025

I think there is also need to fix "env" versions of these settings:

  • docker-compose/docker-compose.yml and docker-compose/docker-compose-hcd.yml have entries like:

    • - STARGATE_JSONAPI_OPERATIONS_DATABASE_CONFIG_CASSANDRA_END_POINTS=hcd

Good point, thanks! Fixed.

Copy link
Contributor

@amorton amorton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approved, we can add a comment and ticket about something we are not changing, see comments.

import io.smallrye.config.WithDefault;

@ConfigMapping(prefix = "stargate.jsonapi.http")
@ConfigMapping(prefix = "data-api.http")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we add a comment in ApiConstants that we are leaving the constant as is there and create a ticket to remove the OfflineMode complely, I dont think we use it and we need to check that. It was for the sstable loader.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 8, 2025

Code Coverage Report

Overall Project 50.43% 🍏
File Coverage
JsonApiMetricsConfig.java 100% 🍏
HeaderBasedSecurityConfiguration.java 100% 🍏
EmbeddingProvidersConfig.java 72.73% 🍏
CreateCollectionOperation.java 67.46% 🍏
TenantConfiguration.java 0% 🍏
DataApiTokenConfiguration.java 0% 🍏

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.

Replace stargate.jsonapi to data-api in the config files

4 participants