From 8f01c22296f540329b05938f16c4699bcd360302 Mon Sep 17 00:00:00 2001
From: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Date: Mon, 16 Sep 2024 22:21:49 +1000
Subject: [PATCH] [8.x] [ES|QL] Enable cursor sync for timeseries charts
 (#192837) (#192982)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[ES|QL] Enable cursor sync for timeseries charts
(#192837)](https://github.com/elastic/kibana/pull/192837)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Stratoula
Kalafateli","email":"efstratia.kalafateli@elastic.co"},"sourceCommit":{"committedDate":"2024-09-16T10:03:58Z","message":"[ES|QL]
Enable cursor sync for timeseries charts (#192837)\n\n##
Summary\r\n\r\nSyncs the cursor for timeseries charts powered by
ES|QL\r\n\r\n\r\n![meow](https://github.com/user-attachments/assets/62664b27-ce95-493d-863a-b5ecaa8006ed)\r\n\r\n\r\n###
Checklist\r\n\r\n- [ ] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios\r\n\r\n---------\r\n\r\nCo-authored-by: kibanamachine
<42973632+kibanamachine@users.noreply.github.com>\r\nCo-authored-by:
Marco Vettorello
<vettorello.marco@gmail.com>","sha":"a581087f530ce4a9143d54d76147b7b645a39da4","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:enhancement","Team:Visualizations","Feature:Lens","v9.0.0","backport:prev-minor","Feature:ES|QL","v8.16.0"],"title":"[ES|QL]
Enable cursor sync for timeseries
charts","number":192837,"url":"https://github.com/elastic/kibana/pull/192837","mergeCommit":{"message":"[ES|QL]
Enable cursor sync for timeseries charts (#192837)\n\n##
Summary\r\n\r\nSyncs the cursor for timeseries charts powered by
ES|QL\r\n\r\n\r\n![meow](https://github.com/user-attachments/assets/62664b27-ce95-493d-863a-b5ecaa8006ed)\r\n\r\n\r\n###
Checklist\r\n\r\n- [ ] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios\r\n\r\n---------\r\n\r\nCo-authored-by: kibanamachine
<42973632+kibanamachine@users.noreply.github.com>\r\nCo-authored-by:
Marco Vettorello
<vettorello.marco@gmail.com>","sha":"a581087f530ce4a9143d54d76147b7b645a39da4"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/192837","number":192837,"mergeCommit":{"message":"[ES|QL]
Enable cursor sync for timeseries charts (#192837)\n\n##
Summary\r\n\r\nSyncs the cursor for timeseries charts powered by
ES|QL\r\n\r\n\r\n![meow](https://github.com/user-attachments/assets/62664b27-ce95-493d-863a-b5ecaa8006ed)\r\n\r\n\r\n###
Checklist\r\n\r\n- [ ] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios\r\n\r\n---------\r\n\r\nCo-authored-by: kibanamachine
<42973632+kibanamachine@users.noreply.github.com>\r\nCo-authored-by:
Marco Vettorello
<vettorello.marco@gmail.com>","sha":"a581087f530ce4a9143d54d76147b7b645a39da4"}},{"branch":"8.x","label":"v8.16.0","branchLabelMappingKey":"^v8.16.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Stratoula Kalafateli <efstratia.kalafateli@elastic.co>
---
 .../active_cursor/active_cursor_utils.test.ts | 33 +++++++++++++++++++
 .../active_cursor/active_cursor_utils.ts      | 11 +++++++
 src/plugins/charts/tsconfig.json              |  1 +
 3 files changed, 45 insertions(+)

diff --git a/src/plugins/charts/public/services/active_cursor/active_cursor_utils.test.ts b/src/plugins/charts/public/services/active_cursor/active_cursor_utils.test.ts
index 31f139eb41a4f..65b6ac7c408ee 100644
--- a/src/plugins/charts/public/services/active_cursor/active_cursor_utils.test.ts
+++ b/src/plugins/charts/public/services/active_cursor/active_cursor_utils.test.ts
@@ -137,6 +137,39 @@ describe('active_cursor_utils', () => {
           }
         `);
       });
+
+      test('should return isDateHistogram true in case the datatable is powered by ES|QL', () => {
+        expect(
+          parseSyncOptions({
+            datatables: [
+              {
+                columns: [
+                  {
+                    id: 'timestamp',
+                    meta: {
+                      type: 'date',
+                    },
+                  },
+                  {
+                    id: 'count',
+                    meta: {
+                      type: 'number',
+                    },
+                  },
+                ],
+                meta: {
+                  type: 'es_ql',
+                },
+              },
+            ] as unknown as Datatable[],
+          })
+        ).toMatchInlineSnapshot(`
+          Object {
+            "accessors": Array [],
+            "isDateHistogram": true,
+          }
+        `);
+      });
     });
   });
 });
diff --git a/src/plugins/charts/public/services/active_cursor/active_cursor_utils.ts b/src/plugins/charts/public/services/active_cursor/active_cursor_utils.ts
index c2126e5efdc2b..8c7fd434a5c08 100644
--- a/src/plugins/charts/public/services/active_cursor/active_cursor_utils.ts
+++ b/src/plugins/charts/public/services/active_cursor/active_cursor_utils.ts
@@ -10,6 +10,7 @@
 import { uniq } from 'lodash';
 
 import type { Datatable } from '@kbn/expressions-plugin/public';
+import { ESQL_TABLE_TYPE } from '@kbn/data-plugin/common';
 import type { ActiveCursorSyncOption, DateHistogramSyncOption } from './types';
 import type { ActiveCursorPayload } from './types';
 
@@ -20,6 +21,16 @@ function isDateHistogramSyncOption(
 }
 
 const parseDatatable = (dataTables: Datatable[]) => {
+  const isEsqlMode = dataTables.some((t) => t?.meta?.type === ESQL_TABLE_TYPE);
+
+  if (isEsqlMode) {
+    return {
+      isDateHistogram:
+        Boolean(dataTables.length) &&
+        dataTables.every((t) => t.columns.some((c) => c.meta.type === 'date')),
+      accessors: [],
+    };
+  }
   const isDateHistogram =
     Boolean(dataTables.length) &&
     dataTables.every((dataTable) =>
diff --git a/src/plugins/charts/tsconfig.json b/src/plugins/charts/tsconfig.json
index 42bbe987f45fa..8ad33a8517031 100644
--- a/src/plugins/charts/tsconfig.json
+++ b/src/plugins/charts/tsconfig.json
@@ -15,6 +15,7 @@
     "@kbn/ui-theme",
     "@kbn/shared-ux-utility",
     "@kbn/config-schema",
+    "@kbn/data-plugin",
   ],
   "exclude": [
     "target/**/*",