iframe-proxy

ykmr1224 · 2026-01-20T17:58:06Z

Description

Extend spath command to support dynamic fields, where it no longer requires field selection.
- It has limitations as it does not fully support field ordering of dynamic fields.
This change is for step2-1 in [RFC] Support Schema-on-Read in PPL #4984
The fields not explicitly referred in the query would be stored in _MAP(dynamic fields), and expanded when result is organized.
It does not support partial wildcard like prefix*, *suffix, and raise error in case those are used.
When result contains dynamic fields, OpenSearchExecutionEngine puts dynamic fields at the end of the fields when organizing result. (This is temporal behavior until field ordering is implemented)
- To align this behavior, it raises error when fields command specify * in the middle of field list.
Following commands will be adopted in later PR (command with multiple input requires change) : appendcol, multisearch, lookup

Related Issues

Check List

New functionality includes testing.
New functionality has been documented.
New functionality has javadoc added.
New functionality has a user manual doc added.
New PPL command checklist all confirmed.
API changes companion pull request created.
Commits are signed per the DCO using --signoff or -s.
Public documentation issue/PR created.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

coderabbitai · 2026-01-20T17:58:21Z

📝 Walkthrough

Summary by CodeRabbit

New Features
- Auto extraction for SPATH that appends conflicting fields and expands dynamic fields into result columns; improved join behavior with configurable overwrite semantics
Documentation
- SPATH docs renamed/expanded to “Auto Extraction”; adds system column guidance, examples, limitations and performance notes
Bug Fixes
- Stronger wildcard/partial-wildcard validation, fillnull safeguards, and more consistent field-resolution across project/aggregation/join flows
Tests
- Large expansion of SPATH and dynamic-field unit and integration tests, including explain and join scenarios

_{✏️ Tip: You can customize this high-level summary in your review settings.}

Walkthrough

Adds end-to-end dynamic-fields (auto extraction): SPATH/field-resolution updates, Calcite planning/merge changes to emit a runtime _MAP, utilities to align/merge dynamic maps during joins, runtime expansion of _MAP into STRING columns, many tests, and docs updates.

Changes

Cohort / File(s)	Summary
Debug util `\`common/src/main/java/org/opensearch/sql/common/utils/DebugUtils.java``	Removed Log4j logger; debug output now uses `System.out.println`.
Field resolution API `\`core/src/main/java/org/opensearch/sql/ast/analysis/FieldResolutionResult.java``	Switched constructors/inputs to `Collection<String>`, added accessors (`getRegularFieldsUnmodifiable`, `hasWildcards`, `hasPartialWildcards`, `hasRegularFields`), set ops (`exclude`, `or`, `and`), and helpers.
Field resolution visitor & SPATH `\`core/src/main/java/org/opensearch/sql/ast/analysis/FieldResolutionVisitor.java``,` `ppl/src/test/java/org/opensearch/sql/ppl/parser/FieldResolutionVisitorTest.java``	Introduced `ALL_FIELDS` sentinel, changed requirement propagation, partial-wildcard checks, added ALL_FIELDS exclusion helpers, SPATH validations, and many SPATH-related tests.
Calcite planning & dynamic-fields helper `\`core/src/main/java/org/opensearch/sql/calcite/CalciteRelNodeVisitor.java``,` `core/src/main/java/org/opensearch/sql/calcite/DynamicFieldsHelper.java``	Injects dynamic `_MAP` handling into planning (project/append/join), enforces wildcard placement when dynamic fields present, and provides utilities to detect/align static vs dynamic fields and build MAP_FROM_ARRAYS/MAP_REMOVE projections.
Join merge & overwrite semantics `\`core/src/main/java/org/opensearch/sql/calcite/utils/JoinAndLookupUtils.java``,` `core/src/main/java/org/opensearch/sql/ast/tree/Join.java``	Added `OverwriteMode` enum and `mergeDynamicFieldsAsNeeded` to merge `_MAP` fields after joins; added `Join.isOverwrite()` accessor and updated rename logic to include DYNAMIC_FIELDS_MAP.
Runtime result expansion `\`core/src/main/java/org/opensearch/sql/calcite/utils/DynamicFieldsResultProcessor.java``,` `opensearch/src/main/java/org/opensearch/sql/opensearch/executor/OpenSearchExecutionEngine.java``	New processor expands `_MAP` into concrete STRING columns and OpenSearchExecutionEngine now post-processes QueryResponse with it.
Constants, Rex/func helpers & functions `\`core/src/main/java/org/opensearch/sql/calcite/plan/DynamicFieldsConstants.java``,` `core/src/main/java/org/opensearch/sql/calcite/ExtendedRexBuilder.java``,` `core/src/main/java/org/opensearch/sql/expression/function/BuiltinFunctionName.java``,` `core/src/main/java/org/opensearch/sql/expression/function/PPLFuncImpTable.java``	Added `DYNAMIC_FIELDS_MAP` constant, ExtendedRexBuilder helpers (`itemCall`, `makeCall`), registered `MAP_FROM_ARRAYS` and operator mapping.
Tests & expected plans `\`core/src/test/... ``,` `integ-test/src/test/... ``,` `integ-test/src/test/resources/expectedOutput/... ``	Many new/updated unit and integration tests covering SPATH, dynamic fields, joins, explains, and DynamicFieldsResultProcessor; new expected explain YAMLs.
Docs & minor tests `\`docs/user/ppl/cmd/spath.md``,` `docs/user/ppl/general/identifiers.md``,` `integ-test/src/test/java/org/opensearch/sql/util/MatcherUtils.java``	Renamed sections to "Auto Extraction", documented `_MAP` system column and wildcard exclusions, improved assertion messages.
Test infra & helpers `\`integ-test/src/test/java/org/opensearch/sql/calcite/remote/CalcitePPLSpathTestBase.java``,` `integ-test/src/test/java/org/opensearch/sql/calcite/remote/CalcitePPLSpathWithJoinIT.java``,` `integ-test/src/test/java/org/opensearch/sql/calcite/remote/CalcitePPLSpathCommandIT.java``	New test base and many integration tests for JSON-backed SPATH scenarios, joins, overwrite behaviors, and dynamic-field interactions.

Sequence Diagram(s)

sequenceDiagram
    autonumber
    participant Client as User Query
    participant FRV as FieldResolutionVisitor
    participant CRV as CalciteRelNodeVisitor
    participant JoinUtil as JoinAndLookupUtils
    participant Exec as OpenSearchExecutionEngine
    participant RP as DynamicFieldsResultProcessor

    Client->>FRV: parse & analyze (detect ALL_FIELDS / wildcards)
    FRV->>CRV: emit Calcite plan with field requirements and `_MAP` markers
    CRV->>JoinUtil: on join -> adjust inputs and request dynamic-map merge
    JoinUtil-->>CRV: merged RelNode with `_MAP` (order per OverwriteMode)
    CRV->>Exec: execute physical plan -> QueryResponse (may include `_MAP`)
    Exec->>RP: expandDynamicFields(QueryResponse)
    RP-->>Client: return expanded QueryResponse (concrete STRING columns)

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~60 minutes

Possibly related PRs

Pushdown join with max=n option to TopHits aggregation #4929 — overlaps CalciteRelNodeVisitor (join handling) and dynamic-field input alignment.
Introduce logical dedup operators for PPL #5014 — related Calcite planning infra changes and metadata-field handling.

Suggested reviewers

penghuo
anirudha
kavithacm
derek-ho
joshuali925
GumpacG
Swiddis
qianheng-aws
LantaoJin

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 21.21% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The PR title 'Support spath with dynamic fields' clearly and concisely describes the main change—extending the spath command to support dynamic fields, which is the primary feature added throughout the changeset.
Description check	✅ Passed	The PR description is directly related to the changeset, detailing the spath dynamic fields feature, its limitations, scope (step 2-1 of issue `#4984`), and implementation specifics including the _MAP field, error handling, and future work.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

📝 Generate docstrings

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Signed-off-by: Tomoyuki Morita <moritato@amazon.com>

dai-chen

Just one concern: I feel it's unclear what future PPL command developers need to understand or implement when adding commands that interact with dynamic fields, since current _MAP implementation seems leaks across multiple layers (field resolution, RelNode visitor, join handling, result post-processing, etc). We can consider improving document or consolidating the logic later if this is valid concern. Thanks!

opensearch-trigger-bot · 2026-01-26T18:50:48Z

* Support spath with dynamic fields Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Add explain test and sql conversion test Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Address comments Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix FieldResolutionResultTest Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Address comment Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix explain Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix join logic to adopt dynamic fields Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix join logic Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix spath.md Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Support fillnull and replace Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Update spath.md Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix test failure Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix doc and error for fillnull Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Accept wildcard only at the end of field list Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Minor fix Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Adopt append command to spath Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix join to allow spath in only one input Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix unit test failure Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix join inputs logic Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix test failure Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Move helper methods Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Add _MAP description in the docs Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Extract more from CalciteRelNodeVisitor Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Refactor IT and address comments Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix FieldResolutionResult Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix DynamicFieldsHelper Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Refactor DynamicFieldsHelper Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix DynamicFieldsHelper Signed-off-by: Tomoyuki Morita <moritato@amazon.com> --------- Signed-off-by: Tomoyuki Morita <moritato@amazon.com> (cherry picked from commit 633d760) Signed-off-by: Tomoyuki Morita <moritato@amazon.com>

* Support spath with dynamic fields Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Add explain test and sql conversion test Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Address comments Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix FieldResolutionResultTest Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Address comment Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix explain Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix join logic to adopt dynamic fields Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix join logic Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix spath.md Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Support fillnull and replace Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Update spath.md Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix test failure Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix doc and error for fillnull Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Accept wildcard only at the end of field list Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Minor fix Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Adopt append command to spath Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix join to allow spath in only one input Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix unit test failure Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix join inputs logic Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix test failure Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Move helper methods Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Add _MAP description in the docs Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Extract more from CalciteRelNodeVisitor Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Refactor IT and address comments Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix FieldResolutionResult Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix DynamicFieldsHelper Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Refactor DynamicFieldsHelper Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix DynamicFieldsHelper Signed-off-by: Tomoyuki Morita <moritato@amazon.com> --------- Signed-off-by: Tomoyuki Morita <moritato@amazon.com>

* Support spath with dynamic fields (#5058) * Support spath with dynamic fields Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Add explain test and sql conversion test Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Address comments Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix FieldResolutionResultTest Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Address comment Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix explain Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix join logic to adopt dynamic fields Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix join logic Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix spath.md Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Support fillnull and replace Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Update spath.md Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix test failure Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix doc and error for fillnull Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Accept wildcard only at the end of field list Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Minor fix Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Adopt append command to spath Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix join to allow spath in only one input Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix unit test failure Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix join inputs logic Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix test failure Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Move helper methods Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Add _MAP description in the docs Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Extract more from CalciteRelNodeVisitor Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Refactor IT and address comments Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix FieldResolutionResult Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix DynamicFieldsHelper Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Refactor DynamicFieldsHelper Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix DynamicFieldsHelper Signed-off-by: Tomoyuki Morita <moritato@amazon.com> --------- Signed-off-by: Tomoyuki Morita <moritato@amazon.com> (cherry picked from commit 633d760) Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix for Java 11 Signed-off-by: Tomoyuki Morita <moritato@amazon.com> --------- Signed-off-by: Tomoyuki Morita <moritato@amazon.com>

* Support spath with dynamic fields Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Add explain test and sql conversion test Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Address comments Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix FieldResolutionResultTest Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Address comment Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix explain Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix join logic to adopt dynamic fields Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix join logic Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix spath.md Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Support fillnull and replace Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Update spath.md Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix test failure Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix doc and error for fillnull Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Accept wildcard only at the end of field list Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Minor fix Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Adopt append command to spath Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix join to allow spath in only one input Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix unit test failure Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix join inputs logic Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix test failure Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Move helper methods Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Add _MAP description in the docs Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Extract more from CalciteRelNodeVisitor Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Refactor IT and address comments Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix FieldResolutionResult Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix DynamicFieldsHelper Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Refactor DynamicFieldsHelper Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Fix DynamicFieldsHelper Signed-off-by: Tomoyuki Morita <moritato@amazon.com> --------- Signed-off-by: Tomoyuki Morita <moritato@amazon.com>

This reverts commit 633d760. Signed-off-by: Chen Dai <daichen@amazon.com>

* Revert "Adopt appendcol, appendpipe, multisearch to spath (#5075)" This reverts commit 7630db8. Signed-off-by: Chen Dai <daichen@amazon.com> * Revert "Support spath with dynamic fields (#5058)" This reverts commit 633d760. Signed-off-by: Chen Dai <daichen@amazon.com> * Revert "Implement spath command with field resolution (#5028)" This reverts commit 65baa2a. Signed-off-by: Chen Dai <daichen@amazon.com> * Fix failed IT testSpathWithMvCombine Signed-off-by: Chen Dai <daichen@amazon.com> --------- Signed-off-by: Chen Dai <daichen@amazon.com>

* Revert "Adopt appendcol, appendpipe, multisearch to spath (opensearch-project#5075)" This reverts commit 7630db8. Signed-off-by: Chen Dai <daichen@amazon.com> * Revert "Support spath with dynamic fields (opensearch-project#5058)" This reverts commit 633d760. Signed-off-by: Chen Dai <daichen@amazon.com> * Revert "Implement spath command with field resolution (opensearch-project#5028)" This reverts commit 65baa2a. Signed-off-by: Chen Dai <daichen@amazon.com> * Fix failed IT testSpathWithMvCombine Signed-off-by: Chen Dai <daichen@amazon.com> --------- Signed-off-by: Chen Dai <daichen@amazon.com>

ykmr1224 self-assigned this Jan 20, 2026

ykmr1224 added enhancement New feature or request PPL Piped processing language calcite calcite migration releated labels Jan 20, 2026

ykmr1224 force-pushed the spath/dynamic-fields-limited branch from d1a62c4 to 3defb88 Compare January 20, 2026 18:28

Support spath with dynamic fields

2e50f6d

Signed-off-by: Tomoyuki Morita <moritato@amazon.com>

ykmr1224 force-pushed the spath/dynamic-fields-limited branch from 3defb88 to 2e50f6d Compare January 20, 2026 19:46

ykmr1224 marked this pull request as ready for review January 20, 2026 20:55

dai-chen approved these changes Jan 26, 2026

View reviewed changes

ykmr1224 merged commit 633d760 into opensearch-project:main Jan 26, 2026
37 checks passed

ykmr1224 added the backport 2.19-dev label Jan 26, 2026

opensearch-trigger-bot Bot added the backport-failed label Jan 26, 2026

ykmr1224 mentioned this pull request Jan 26, 2026

[Backport 2.19-dev] Support spath with dynamic fields (#5058) #5073

Merged

coderabbitai Bot mentioned this pull request Jan 26, 2026

Adding PPL convert command with numeric and memory conversion functions #5034

Closed

8 tasks

LantaoJin added the backport-manually Filed a PR to backport manually. label Jan 27, 2026

coderabbitai Bot mentioned this pull request Jan 27, 2026

Feature/mvcombine #5025

Merged

coderabbitai Bot mentioned this pull request Jan 27, 2026

Adopt appendcol, appendpipe, multisearch to spath #5075

Merged

8 tasks

This was referenced Jan 27, 2026

[Feature] fieldformat command implementation #5080

Merged

Use Calcite's validation system for type checking & coercion #4892

Open

Add validation for expand command on scalar types (#5065) #5089

Closed

This was referenced Feb 4, 2026

Mvexpand feature #4944

Closed

Add nomv command #5130

Merged

dai-chen mentioned this pull request Feb 10, 2026

(Calcite PPL) spath command dynamic columns support #4307

Closed

dai-chen added a commit to dai-chen/sql-1 that referenced this pull request Feb 11, 2026

Revert "Support spath with dynamic fields (opensearch-project#5058)"

bd432c7

This reverts commit 633d760. Signed-off-by: Chen Dai <daichen@amazon.com>

dai-chen mentioned this pull request Feb 11, 2026

Revert dynamic column support #5139

Merged

8 tasks

dai-chen added a commit to dai-chen/sql-1 that referenced this pull request Feb 11, 2026

Revert "Support spath with dynamic fields (opensearch-project#5058)"

4a3d439

This reverts commit 633d760. Signed-off-by: Chen Dai <daichen@amazon.com>

This was referenced Feb 13, 2026

Add auto-extract mode for spath command #5140

Merged

[Feature] PPL Command: MvExpand #5144

Merged

Fix path navigation on map columns for spath command #5149

Merged

[Feature] Calcite PPL search result highlighting #5141

Closed

coderabbitai Bot mentioned this pull request Feb 21, 2026

feat: Implement PPL convert command with 5 conversion functions #5157

Merged

8 tasks

Sunbelt Computer Software

PL/B Language Development and Support

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Support spath with dynamic fields#5058

Support spath with dynamic fields#5058
ykmr1224 merged 28 commits into
opensearch-project:mainfrom
ykmr1224:spath/dynamic-fields-limited

ykmr1224 commented Jan 20, 2026 •

edited

Loading

Uh oh!

coderabbitai Bot commented Jan 20, 2026 •

edited

Loading

Summary by CodeRabbit

Walkthrough

Changes

Sequence Diagram(s)

Estimated code review effort

Possibly related PRs

Suggested reviewers

Uh oh!

dai-chen left a comment

Uh oh!

Uh oh!

opensearch-trigger-bot Bot commented Jan 26, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Sunbelt Computer Software

PL/B Language Development and Support

Uh oh!

Conversation

ykmr1224 commented Jan 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Related Issues

Check List

Uh oh!

coderabbitai Bot commented Jan 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Walkthrough

Changes

Sequence Diagram(s)

Estimated code review effort

Possibly related PRs

Suggested reviewers

Uh oh!

dai-chen left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

opensearch-trigger-bot Bot commented Jan 26, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

ykmr1224 commented Jan 20, 2026 •

edited

Loading

coderabbitai Bot commented Jan 20, 2026 •

edited

Loading