Skip to content

Commit b7cf1f3

Browse files
committed
Fix bug of partial pipeline not bucketing normalization processor correctly
Signed-off-by: Tyler Ohlsen <ohltyler@amazon.com>
1 parent 545ae3d commit b7cf1f3

File tree

1 file changed

+16
-1
lines changed

1 file changed

+16
-1
lines changed

public/utils/form_to_pipeline_utils.ts

+16-1
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ import {
88
IProcessorConfig,
99
IngestPipelineConfig,
1010
PROCESSOR_CONTEXT,
11+
PROCESSOR_TYPE,
1112
SearchPipelineConfig,
1213
WorkflowConfig,
1314
WorkflowFormValues,
@@ -83,8 +84,17 @@ export function formikToPartialPipeline(
8384
requestProcessors,
8485
context
8586
),
87+
// for search response, we need to explicitly separate out any phase results processors
88+
phase_results_processors: processorConfigsToTemplateProcessors(
89+
precedingProcessors.filter((processor) =>
90+
isPhaseResultsProcessor(processor)
91+
),
92+
context
93+
),
8694
response_processors: processorConfigsToTemplateProcessors(
87-
precedingProcessors,
95+
precedingProcessors.filter(
96+
(processor) => !isPhaseResultsProcessor(processor)
97+
),
8898
context
8999
),
90100
} as SearchPipelineConfig)
@@ -113,3 +123,8 @@ function getPrecedingProcessors(
113123
});
114124
return precedingProcessors;
115125
}
126+
127+
// currently the only phase results processor supported is the normalization processor
128+
function isPhaseResultsProcessor(processor: IProcessorConfig): boolean {
129+
return processor.type === PROCESSOR_TYPE.NORMALIZATION;
130+
}

0 commit comments

Comments
 (0)