[Uptime] Simplify snapshot max to Infinity (#55931) (#56064)

Fixes elastic/uptime#119 Rather than relying on a contant for the max number of monitors, it's easier to just use infinity. This is simpler than making the iterator more complex with an 'infinite' mode. Co-authored-by: Elastic Machine <[email protected]>
elastic · Jan 28, 2020 · 0c5c054 · 0c5c054
1 parent 617dbab
commit 0c5c054
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 7 deletions.
diff --git a/x-pack/legacy/plugins/uptime/common/constants/context_defaults.ts b/x-pack/legacy/plugins/uptime/common/constants/context_defaults.ts
@@ -19,10 +19,4 @@ export const CONTEXT_DEFAULTS = {
     cursorDirection: CursorDirection.AFTER,
     sortOrder: SortOrder.ASC,
   },
-
-  /**
-   * Defines the maximum number of monitors to iterate on
-   * in a single count session. The intention is to catch as many as possible.
-   */
-  MAX_MONITORS_FOR_SNAPSHOT_COUNT: 1000000,
 };
diff --git a/...plugins/uptime/server/lib/adapters/monitor_states/elasticsearch_monitor_states_adapter.ts b/...plugins/uptime/server/lib/adapters/monitor_states/elasticsearch_monitor_states_adapter.ts
@@ -61,7 +61,7 @@ export const elasticsearchMonitorStatesAdapter: UMMonitorStatesAdapter = {
       dateRangeEnd,
       CONTEXT_DEFAULTS.CURSOR_PAGINATION,
       filters && filters !== '' ? JSON.parse(filters) : null,
-      CONTEXT_DEFAULTS.MAX_MONITORS_FOR_SNAPSHOT_COUNT,
+      Infinity,
       statusFilter
     );