[ML] Fixing exclude frequent in advanced wizard (#81121)

* [ML] Fixing exclude frequent in advanced wizard * updating description * adding exclude_frequent to test
elastic · Oct 20, 2020 · 6db108b · 6db108b
1 parent 8162b96
commit 6db108b
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 3 deletions.
diff --git a/...omponents/pick_fields_step/components/advanced_detector_modal/advanced_detector_modal.tsx b/...omponents/pick_fields_step/components/advanced_detector_modal/advanced_detector_modal.tsx
@@ -58,7 +58,12 @@ const emptyOption: EuiComboBoxOptionOption = {
   label: '',
 };
 
-const excludeFrequentOptions: EuiComboBoxOptionOption[] = [{ label: 'all' }, { label: 'none' }];
+const excludeFrequentOptions: EuiComboBoxOptionOption[] = [
+  { label: 'all' },
+  { label: 'none' },
+  { label: 'by' },
+  { label: 'over' },
+];
 
 export const AdvancedDetectorModal: FC<Props> = ({
   payload,

diff --git a/...job/pages/components/pick_fields_step/components/advanced_detector_modal/descriptions.tsx b/...job/pages/components/pick_fields_step/components/advanced_detector_modal/descriptions.tsx
@@ -142,7 +142,7 @@ export const ExcludeFrequentDescription: FC = memo(({ children }) => {
       description={
         <FormattedMessage
           id="xpack.ml.newJob.wizard.pickFieldsStep.advancedDetectorModal.excludeFrequent.description"
-          defaultMessage="If true will automatically identify and exclude frequently occurring entities which may otherwise have dominated results."
+          defaultMessage="If set, it will automatically identify and exclude frequently occurring entities which may otherwise have dominated results."
         />
       }
     >

diff --git a/x-pack/plugins/ml/server/routes/schemas/anomaly_detectors_schema.ts b/x-pack/plugins/ml/server/routes/schemas/anomaly_detectors_schema.ts
@@ -26,6 +26,8 @@ const detectorSchema = schema.object({
   over_field_name: schema.maybe(schema.string()),
   partition_field_name: schema.maybe(schema.string()),
   detector_description: schema.maybe(schema.string()),
+  exclude_frequent: schema.maybe(schema.string()),
+  use_null: schema.maybe(schema.boolean()),
   /** Custom rules */
   custom_rules: customRulesSchema,
 });

diff --git a/x-pack/test/api_integration/apis/ml/job_validation/validate.ts b/x-pack/test/api_integration/apis/ml/job_validation/validate.ts
@@ -35,7 +35,13 @@ export default ({ getService }: FtrProviderContext) => {
           groups: [],
           analysis_config: {
             bucket_span: '15m',
-            detectors: [{ function: 'mean', field_name: 'products.discount_amount' }],
+            detectors: [
+              {
+                function: 'mean',
+                field_name: 'products.discount_amount',
+                exclude_frequent: 'none',
+              },
+            ],
             influencers: [],
             summary_count_field_name: 'doc_count',
           },