@cumulus/db 20.0.0 → 20.0.2

package/dist/search/StatsSearch.js

@@ -24,9 +24,8 @@ const infixMapping = {
 class StatsSearch extends BaseSearch_1.BaseSearch {
     constructor(event, type) {
         const { field, ...queryStringParameters } = event.queryStringParameters || {};
-        super({ queryStringParameters }, type);
+        super({ queryStringParameters: { ...queryStringParameters, limit: 'null' } }, type);
         this.field = field ?? 'status';
-        this.dbQueryParameters = (0, omit_1.default)(this.dbQueryParameters, ['limit', 'offset']);
     }
     /**
      * Formats the postgres records into an API stats/aggregate response

package/dist/search/queries.js

@@ -33,6 +33,13 @@ const regexes = {
     exists: /^(.*)__exists$/,
     range: /^(.*)__(from|to)$/,
 };
+/**
+ * Based on PostgreSQL documentation, when using LIMIT, it is important to use
+ * an ORDER BY clause that constrains the result rows into a unique order.
+ * The following is the default sort column and order for the pagination.
+ */
+const defaultSortColumn = 'cumulus_id';
+const defaultSortOrder = 'asc';
 /**
  * Convert 'exists' query fields to db query parameters from api query string fields
  *
@@ -166,7 +173,7 @@ const convertTerms = (type, queryStringFields) => {
  */
 const convertSort = (type, queryStringParameters) => {
     const sortArray = [];
-    const { sort_by: sortBy, sort_key: sortKey } = queryStringParameters;
+    const { sort_by: sortBy, sort_key: sortKey, limit } = queryStringParameters;
     let { order } = queryStringParameters;
     if (sortBy) {
         order = order ?? 'asc';
@@ -180,6 +187,9 @@ const convertSort = (type, queryStringParameters) => {
             return Object.keys(queryParam ?? {}).map((key) => sortArray.push({ column: key, order }));
         });
     }
+    if (limit !== 'null') {
+        sortArray.push({ column: defaultSortColumn, order: defaultSortOrder });
+    }
     return sortArray;
 };
 /**
@@ -204,7 +214,7 @@ const convertQueryStringToDbQueryParameters = (type, queryStringParameters) => {
     const { limit, page, prefix, infix, fields, estimateTableRowCount, includeFullRecord, } = queryStringParameters;
     const dbQueryParameters = {};
     dbQueryParameters.page = Number.parseInt(page ?? '1', 10);
-    if (limit !== null) {
+    if (limit !== 'null') {
         dbQueryParameters.limit = Number.parseInt(limit ?? '10', 10);
         dbQueryParameters.offset = (dbQueryParameters.page - 1) * dbQueryParameters.limit;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cumulus/db",
-  "version": "20.0.0",
+  "version": "20.0.2",
   "description": "Utilities for working with the Cumulus DB",
   "license": "Apache-2.0",
   "main": "./dist/index.js",
@@ -33,12 +33,12 @@
   },
   "dependencies": {
     "@aws-sdk/client-secrets-manager": "^3.621.0",
-    "@cumulus/aws-client": "20.0.0",
-    "@cumulus/common": "20.0.0",
-    "@cumulus/errors": "20.0.0",
-    "@cumulus/logger": "20.0.0",
-    "@cumulus/message": "20.0.0",
-    "@cumulus/types": "20.0.0",
+    "@cumulus/aws-client": "20.0.2",
+    "@cumulus/common": "20.0.2",
+    "@cumulus/errors": "20.0.2",
+    "@cumulus/logger": "20.0.2",
+    "@cumulus/message": "20.0.2",
+    "@cumulus/types": "20.0.2",
     "crypto-random-string": "^3.2.0",
     "is-valid-hostname": "1.0.2",
     "knex": "2.4.1",
@@ -50,5 +50,5 @@
   "devDependencies": {
     "@types/uuid": "^8.0.0"
   },
-  "gitHead": "d43b620da63c0dc7076b9e619ea6d9b92204fcaa"
+  "gitHead": "f0086d4fb4460f2f17835f91903b5e7918376806"
 }