TSVB is retaining results even after indexpattern is removed from opt…

src/legacy/core_plugins/metrics/public/components/index_pattern.js

@@ -44,6 +44,7 @@ export const IndexPattern = props => {
   const dropBucketName = `${prefix}drop_last_bucket`;
 
   const defaults = {
+    default_index_pattern: '',
     [indexPatternName]: '*',
     [intervalName]: 'auto',
     [dropBucketName]: 1
@@ -62,11 +63,16 @@ export const IndexPattern = props => {
               id="tsvb.indexPatternLabel"
               defaultMessage="Index pattern"
             />)}
+            helpText={(model.default_index_pattern && !model[indexPatternName] && <FormattedMessage
+              id="tsvb.indexPattern.searchByDefaultIndex"
+              defaultMessage="Default index pattern is used. To query all indexes use *"
+            />)}
             fullWidth
           >
             <EuiFieldText
               data-test-subj="metricsIndexPatternInput"
               disabled={props.disabled}
+              placeholder={model.default_index_pattern}
               onChange={handleTextChange(indexPatternName, '*')}
               value={model[indexPatternName]}
               fullWidth

src/legacy/core_plugins/metrics/public/components/vis_editor.js

@@ -72,13 +72,12 @@ class VisEditor extends Component {
   }
 
   setDefaultIndexPattern = async () => {
-    if (this.props.vis.params.index_pattern === '') {
-      // set the default index pattern if none is defined.
-      const savedObjectsClient = chrome.getSavedObjectsClient();
-      const indexPattern = await savedObjectsClient.get('index-pattern', this.getConfig('defaultIndex'));
-      const defaultIndexPattern = indexPattern.attributes.title;
-      this.props.vis.params.index_pattern = defaultIndexPattern;
-    }
+    const savedObjectsClient = chrome.getSavedObjectsClient();
+    const indexPattern = await savedObjectsClient.get('index-pattern', this.getConfig('defaultIndex'));
+
+    this.handleChange({
+      default_index_pattern: indexPattern.attributes.title
+    });
   }
 
   handleChange = async (partialModel) => {

src/legacy/core_plugins/metrics/server/lib/get_fields.js

@@ -18,13 +18,15 @@
  */
 import { SearchStrategiesRegister } from './search_strategies/search_strategies_register';
 import { uniq } from 'lodash';
+import { getIndexPatternObject } from './vis_data/helpers/get_index_pattern';
 
 export async function getFields(req) {
-  const indexPattern = req.query.index || '*';
-  const { searchStrategy, capabilities } = await SearchStrategiesRegister.getViableStrategy(req, indexPattern);
+  const indexPattern = req.query.index;
+  const { indexPatternString } = await getIndexPatternObject(req, indexPattern);
+  const { searchStrategy, capabilities } = await SearchStrategiesRegister.getViableStrategy(req, indexPatternString);
 
   const fields = (await searchStrategy
-    .getFieldsForWildcard(req, indexPattern, capabilities))
+    .getFieldsForWildcard(req, indexPatternString, capabilities))
     .filter(field => field.aggregatable);
 
   return uniq(fields, field => field.name);

src/legacy/core_plugins/metrics/server/lib/vis_data/helpers/get_index_pattern.js

@@ -16,6 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
+import { get } from  'lodash';
 
 export async function getIndexPatternObject(req, indexPatternString) {
   let defaultIndex;
@@ -48,6 +49,6 @@ export async function getIndexPatternObject(req, indexPatternString) {
 
   return {
     indexPatternObject,
-    indexPatternString: indexPatternString || indexPatternObject.title
+    indexPatternString: indexPatternString || get(indexPatternObject, 'title', '')
   };
 }