Add default size for custom vector map

Signed-off-by: Vijayan Balasubramanian <balasvij@amazon.com>
opensearch-project · Feb 9, 2023 · 3300cbb · 3300cbb
1 parent d9dc91d
commit 3300cbb
Show file tree

Hide file tree

Showing 5 changed files with 11 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -93,6 +93,7 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 - [VisBuilder] Fixes pipeline aggs ([#3137](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/3137))
 - [Region Maps] Fixes bug that prevents selected join field to be used ([#3213](Fix bug that prevents selected join field to be used))
 - [Multi DataSource]Update test connection button text([#3247](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/3247))
+- [Region Maps] Add default size for custom vector map([#3399](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/3399))
 
 ### 🚞 Infrastructure
 

diff --git a/src/plugins/region_map/common/constants/shared.ts b/src/plugins/region_map/common/constants/shared.ts
@@ -5,3 +5,4 @@
 
 export const DEFAULT_MAP_CHOICE = 'default';
 export const CUSTOM_MAP_CHOICE = 'custom';
+export const DEFAULT_INDEX_FETCH_SIZE = 10000; // opensearch index.max_result_window default value
diff --git a/src/plugins/region_map/common/index.ts b/src/plugins/region_map/common/index.ts
@@ -3,6 +3,10 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-import { DEFAULT_MAP_CHOICE, CUSTOM_MAP_CHOICE } from './constants/shared';
+import {
+  DEFAULT_MAP_CHOICE,
+  CUSTOM_MAP_CHOICE,
+  DEFAULT_INDEX_FETCH_SIZE,
+} from './constants/shared';
 
-export { DEFAULT_MAP_CHOICE, CUSTOM_MAP_CHOICE };
+export { DEFAULT_MAP_CHOICE, CUSTOM_MAP_CHOICE, DEFAULT_INDEX_FETCH_SIZE };
diff --git a/src/plugins/region_map/public/components/map_choice_options.tsx b/src/plugins/region_map/public/components/map_choice_options.tsx
@@ -4,7 +4,7 @@
  */
 
 import './map_choice_options.scss';
-import React, { useCallback, useMemo, useState } from 'react';
+import React, { useCallback, useMemo } from 'react';
 import {
   EuiPanel,
   EuiSpacer,

diff --git a/src/plugins/region_map/server/routes/opensearch.ts b/src/plugins/region_map/server/routes/opensearch.ts
@@ -5,6 +5,7 @@
 
 import { schema } from '@osd/config-schema';
 import { IRouter } from 'opensearch-dashboards/server';
+import { DEFAULT_INDEX_FETCH_SIZE } from '../../common';
 
 export function registerGeospatialRoutes(router: IRouter) {
   router.post(
@@ -64,7 +65,7 @@ export function registerGeospatialRoutes(router: IRouter) {
       const client = context.core.opensearch.client.asCurrentUser;
       try {
         const { index } = req.body;
-        const params = { index, body: {} };
+        const params = { index, body: {}, size: DEFAULT_INDEX_FETCH_SIZE };
         const results = await client.search(params);
         return res.ok({
           body: {
Original file line number	Diff line number	Diff line change
Expand Up		@@ -5,3 +5,4 @@

		export const DEFAULT_MAP_CHOICE = 'default';
		export const CUSTOM_MAP_CHOICE = 'custom';
		export const DEFAULT_INDEX_FETCH_SIZE = 10000; // opensearch index.max_result_window default value