Skip to content

Add metadata to configuration exceptions #27728

Closed
#32322
Closed
Add metadata to configuration exceptions#27728
#32322
Assignees
original-brownbear
Labels
:Distributed/Ingest NodeExecution or management of Ingest Pipelines>enhancementhelp wantedadoptme
@DaveCTurner

Description

@DaveCTurner
DaveCTurner
opened on Dec 8, 2017

... instead of using headers.

Similarly to #27724, there is some exception metadata that is included in headers rather than using addMetadata(), introduced in #19318.

elasticsearch/core/src/main/java/org/elasticsearch/ingest/ConfigurationUtils.java

Lines 346 to 354 in 2e36f19

if (processorType != null) {
exception.addHeader("processor_type", processorType);
}
if (processorTag != null) {
exception.addHeader("processor_tag", processorTag);
}
if (propertyName != null) {
exception.addHeader("property_name", propertyName);
}

Under the umbrella of #27672, it'd be preferable to use addMetadata() to reduce the number of ways that clients have to handle errors.

/cc @simonw and @talevy for further opinions.

Metadata

Metadata

Assignees

Labels

:Distributed/Ingest NodeExecution or management of Ingest Pipelines>enhancementhelp wantedadoptme

Type

No type

Fields

No fields configured for issues without a type.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions