Skip to content

Commit

Permalink
Remove deprecation notice from ascending sort for terms (#23421)
Browse files Browse the repository at this point in the history
  • Loading branch information
lukeelmers authored Sep 25, 2018
1 parent fceddf8 commit 9b6d0b1
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions src/ui/public/agg_types/buckets/terms.js
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,6 @@ import otherBucketTemplate from '../controls/other_bucket.html';

import { getRequestInspectorStats, getResponseInspectorStats } from '../../courier/utils/courier_inspector_utils';
import { buildOtherBucketAgg, mergeOtherBucketAggResponse, updateMissingBucket } from './_terms_other_bucket_helper';
import { toastNotifications } from '../../notify';

const aggFilter = [
'!top_hits', '!percentiles', '!median', '!std_dev',
Expand Down Expand Up @@ -236,9 +235,6 @@ export const termsBucketAgg = new BucketAggType({
}

if (orderAgg.type.name === 'count') {
if (dir === 'asc') {
toastNotifications.addWarning('Sorting in Ascending order by Count in Terms aggregations is deprecated');
}
order._count = dir;
return;
}
Expand Down

0 comments on commit 9b6d0b1

Please sign in to comment.