Skip to content

Use JSON ingestion as schema definition extension#1205

Draft
jwils wants to merge 1 commit into
joshuaw/json-ingestion-extension-modulesfrom
joshuaw/json-ingestion-default-extension
Draft

Use JSON ingestion as schema definition extension#1205
jwils wants to merge 1 commit into
joshuaw/json-ingestion-extension-modulesfrom
joshuaw/json-ingestion-default-extension

Conversation

@jwils
Copy link
Copy Markdown
Collaborator

@jwils jwils commented May 20, 2026

Why

Complete the extraction by loading JSON ingestion as the default schema_definition extension and removing the duplicated core JSON Schema implementation.

What

  • Load elasticgraph-json_ingestion as the default schema definition extension when available
  • Move JSON Schema artifact/results behavior out of core schema_definition
  • Remove JSON Schema field/type/index logic from core classes now provided by the extension
  • Update tests, RBS signatures, and support wiring for the extracted default extension

Risk Assessment

Medium to high - this is the behavior-transfer layer, but it sits after the move/namespace/add-extension PRs so the remaining diff is mostly deletion and wiring.

References

  • Stacked on Add JSON ingestion schema extension modules #1204.
  • Local lint, type check, schema artifact check, dependency diagram verification, and spellcheck passed.
  • script/run_gem_specs elasticgraph-schema_definition could not run locally because no test datastore is running at http://localhost:9234; GitHub CI should cover that environment.

@jwils jwils force-pushed the joshuaw/json-ingestion-default-extension branch from f9fb615 to edc42c3 Compare May 20, 2026 20:03
@jwils jwils force-pushed the joshuaw/json-ingestion-extension-modules branch 2 times, most recently from 024c4ef to d8c3c70 Compare May 20, 2026 20:15
@jwils jwils force-pushed the joshuaw/json-ingestion-default-extension branch from edc42c3 to 62013f0 Compare May 20, 2026 20:15
@jwils jwils force-pushed the joshuaw/json-ingestion-extension-modules branch from d8c3c70 to b78686f Compare May 21, 2026 02:49
@jwils jwils force-pushed the joshuaw/json-ingestion-default-extension branch from 62013f0 to 1eec3a6 Compare May 21, 2026 02:49
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