Skip to content

Remove obsolete cloud_optimized_index configuration - #350

Closed
shaunahu wants to merge 1 commit into
mainfrom
chore/remove-cloud-optimized-index
Closed

shaunahu wants to merge 1 commit into
mainfrom
chore/remove-cloud-optimized-index

Conversation

@shaunahu

Copy link
Copy Markdown
Contributor

What

Removes the elasticsearch.cloud_optimized_index property, which nothing uses any more:

  • Dockerfile: drop -Delasticsearch.cloud_optimized_index.name=${CO_INDEX_NAME}.
  • server/src/main/resources/application.yaml: drop the cloud_optimized_index block.
  • BaseTestClass:
    • drop the two @Value fields bound to it;
    • drop the data_index_schema.json test index;
    • drop insertJsonToElasticCODataIndex(), which has no callers.

Why

  • The es-coindexer-<env> index used to hold cloud-optimised data points for the hexbin layer.
  • PMTiles replaced it (#8430), and es-indexer removed IndexCloudOptimizedService (#8789). Nothing writes the index now.
  • On main, ogcapi-java only bound the property; no production code read it. FeatureId.summary has no handler, so items/summary already returns 501.

Testing

  • ElasticSearchTest (mocked): 10 of 10 pass locally, and the test sources compile.
  • The Testcontainers integration tests (everything extending BaseTestClass) can't start Elasticsearch on my machine, so I'm relying on CI for them. initSchemas() now creates two test indexes instead of three.

Follow-ups (not in this PR)

  • Deploy config: CO_INDEX_NAME may still be set in the deploy config (aodn/appdeploy). That's harmless now, but it can be removed.
  • es-indexer: it still ships stacmodel/src/main/resources/schema/data_index_schema.json, and nothing loads it after this change.

🤖 Generated with Claude Code

The es-coindexer index is no longer written or read: PMTiles replaced it (#8430) and es-indexer
removed IndexCloudOptimizedService (#8789). Only a test fixture still bound the property.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@shaunahu shaunahu closed this Sep 26, 2026
@shaunahu
shaunahu deleted the chore/remove-cloud-optimized-index branch September 26, 2026 03:30

This branch was successfully deployed

1 active deployment
test — a8296fee Deployed Sep 26, 2026 by shaunahu via build_test #844
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