Skip to content

[8.19] Refactor SourceProvider creation to consistently use MappingLookup (#…#128312

Merged
elasticsearchmachine merged 1 commit intoelastic:8.19from
jimczi:backport_128213
May 22, 2025
Merged

[8.19] Refactor SourceProvider creation to consistently use MappingLookup (#…#128312
elasticsearchmachine merged 1 commit intoelastic:8.19from
jimczi:backport_128213

Conversation

@jimczi
Copy link
Contributor

@jimczi jimczi commented May 22, 2025

Backport of #128213 to 8.19.

…lastic#128213)

This change updates the code to always create SourceProvider instances via MappingLookup, avoiding direct exposure to the underlying source format (synthetic or stored).
It also aligns source filtering behaviour between SourceProvider and SourceLoader, ensuring consistent application of filters.

This change is needed to enable source filtering to occur earlier in the fetch phase, for example, when constructing a synthetic source.
@jimczi jimczi added >enhancement backport :StorageEngine/Mapping The storage related side of mappings v8.19.0 auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) labels May 22, 2025
@elasticsearchmachine elasticsearchmachine merged commit c98a57a into elastic:8.19 May 22, 2025
15 checks passed
@jimczi jimczi deleted the backport_128213 branch May 22, 2025 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport >enhancement :StorageEngine/Mapping The storage related side of mappings v8.19.0

2 participants