Refactor popuplarize function

src/plugins/data/public/index_patterns/index_patterns/index_pattern.ts

@@ -298,7 +298,14 @@ export class IndexPattern implements IIndexPattern {
     return this.save();
   }
 
+  /**
+   * This function is just used by Discover and it's high likely to be removed in the near future
+   * It doesn't use the save function to skip the error message that's displayed when
+   * a user adds several columns in a higher frequency that the changes can be persisted to ES
+   * resulting in 409 errors
+   */
   async popularizeField(fieldName: string, unit = 1) {
+    if (!this.id) return;
     const field = this.fields.getByName(fieldName);
     if (!field) {
       return;
@@ -308,7 +315,15 @@ export class IndexPattern implements IIndexPattern {
       return;
     }
     field.count = count;
-    await this.save();
+
+    try {
+      const res = await this.savedObjectsClient.update(type, this.id, this.prepBody(), {
+        version: this.version,
+      });
+      this.version = res._version;
+    } catch (e) {
+      // no need for an error message here
+    }
   }
 
   getNonScriptedFields() {