Skip to content

[Deprecation API] Adjust details in the SourceFieldMapper deprecation warning #122041

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
elasticsearchmachine merged 5 commits into from
Feb 7, 2025
Merged

[Deprecation API] Adjust details in the SourceFieldMapper deprecation warning #122041

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
7ed7655
Adjust details in the SourceFieldMapper deprecation warning
gmarouli Feb 7, 2025
bb38010
Use an alias link to allow us to provide a proper guide soon.
gmarouli Feb 7, 2025
3a90357
Keep the deprecation warning and introduce a title.
gmarouli Feb 7, 2025
2f5affc
Use the correct warning
gmarouli Feb 7, 2025
3947328
Add bwc assertions until the backports are completed.
gmarouli Feb 7, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion server/src/main/java/org/elasticsearch/index/mapper/SourceFieldMapper.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,9 @@ public class SourceFieldMapper extends MetadataFieldMapper {

public static final String LOSSY_PARAMETERS_ALLOWED_SETTING_NAME = "index.lossy.source-mapping-parameters";

public static final String DEPRECATION_WARNING = "Configuring source mode in mappings is deprecated and will be removed "
public static final String DEPRECATION_WARNING = "Configuring source mode in mappings is deprecated.";

public static final String DEPRECATION_DETAILS = "Configuring source mode in mappings is deprecated and will be removed "
Copy link
Contributor

@kkrik-es kkrik-es Feb 7, 2025

Choose a reason for hiding this comment

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

Since this is used in many places, let's keep this as is and add DEPRECATION_WARNING_TITLE above.

Copy link
Contributor Author

@gmarouli gmarouli Feb 7, 2025

Choose a reason for hiding this comment

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

Good point! I adjusted it. I did not notice that this is also added a header warning.

+ "in future versions. Use [index.mapping.source.mode] index setting instead.";

/** The source mode */
Expand Down
2 changes: 1 addition & 1 deletion ...ecation/src/main/java/org/elasticsearch/xpack/deprecation/TemplateDeprecationChecker.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,7 @@ private DeprecationIssue checkSourceModeInComponentTemplates(ComponentTemplate t
DeprecationIssue.Level.CRITICAL,
SourceFieldMapper.DEPRECATION_WARNING,
"https://github.com/elastic/elasticsearch/pull/117172",
null,
SourceFieldMapper.DEPRECATION_DETAILS,
false,
null
);
Expand Down
2 changes: 1 addition & 1 deletion ...on/src/test/java/org/elasticsearch/xpack/deprecation/TemplateDeprecationCheckerTests.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ public void testCheckSourceModeInComponentTemplates() throws IOException {
DeprecationIssue.Level.CRITICAL,
SourceFieldMapper.DEPRECATION_WARNING,
"https://github.com/elastic/elasticsearch/pull/117172",
null,
SourceFieldMapper.DEPRECATION_DETAILS,
false,
null
);
Expand Down