Skip to content

Commit

Permalink
[Discover] Removes project command from our code (elastic#176368)
Browse files Browse the repository at this point in the history
## Summary

This is a cleanup, no need to check for the project command as it is
being removed in 8.13
elastic/elasticsearch#105064

The project command was deprecated even on the first release and we were
showing a warning in the users to use keep instead. As we are still in
tech preview, the ES team decided to remove it (now it will fail) so we
are aligning here and also remove this from our code.
  • Loading branch information
stratoula authored Feb 7, 2024
1 parent c38410a commit c8e6816
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ import { FetchStatus } from '../../types';

const MAX_NUM_OF_COLUMNS = 50;
// For ES|QL we want in case of the following commands to display a table view, otherwise display a document view
const TRANSFORMATIONAL_COMMANDS = ['stats', 'project', 'keep'];
const TRANSFORMATIONAL_COMMANDS = ['stats', 'keep'];

/**
* Hook to take care of text based query language state transformations when a new result is returned
Expand Down
2 changes: 1 addition & 1 deletion src/plugins/unified_histogram/public/layout/helpers.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@

import { AggregateQuery } from '@kbn/es-query';

const TRANSFORMATIONAL_COMMANDS = ['stats', 'project', 'keep'];
const TRANSFORMATIONAL_COMMANDS = ['stats', 'keep'];

export const shouldDisplayHistogram = (query: AggregateQuery) => {
let queryHasTransformationalCommands = false;
Expand Down

0 comments on commit c8e6816

Please sign in to comment.