[ML] Fixing file data visualizer override arguments (elastic#60627)

jgowdyelastic · Mar 19, 2020 · f4e7268 · f4e7268
1 parent 7a13bb2
commit f4e7268
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/x-pack/plugins/ml/public/application/datavisualizer/file_based/components/utils/utils.js b/x-pack/plugins/ml/public/application/datavisualizer/file_based/components/utils/utils.js
@@ -66,6 +66,10 @@ export function createUrlOverrides(overrides, originalSettings) {
     ) {
       formattedOverrides.format = originalSettings.format;
     }
+
+    if (Array.isArray(formattedOverrides.column_names)) {
+      formattedOverrides.column_names = formattedOverrides.column_names.join();
+    }
   }
 
   if (formattedOverrides.format === '' && originalSettings.format === 'semi_structured_text') {
@@ -82,11 +86,6 @@ export function createUrlOverrides(overrides, originalSettings) {
     formattedOverrides.column_names = '';
   }
 
-  // escape grok pattern as it can contain bad characters
-  if (formattedOverrides.grok_pattern !== '') {
-    formattedOverrides.grok_pattern = encodeURIComponent(formattedOverrides.grok_pattern);
-  }
-
   if (formattedOverrides.lines_to_sample === '') {
     formattedOverrides.lines_to_sample = overrides.linesToSample;
   }