Merge pull request #1481 from bluewave-labs/fix/be/db-indices

fix: be/db indices

Server/db/models/Check.js

@@ -67,5 +67,8 @@ const BaseCheckSchema = mongoose.Schema({
  */
 const CheckSchema = mongoose.Schema({ ...BaseCheckSchema.obj }, { timestamps: true });
 CheckSchema.index({ createdAt: 1 });
+CheckSchema.index({ monitorId: 1, createdAt: 1 });
+CheckSchema.index({ monitorId: 1, createdAt: -1 });
+
 export default mongoose.model("Check", CheckSchema);
 export { BaseCheckSchema };

Server/db/models/HardwareCheck.js

@@ -64,5 +64,7 @@ const HardwareCheckSchema = mongoose.Schema(
 );
 
 HardwareCheckSchema.index({ createdAt: 1 });
+HardwareCheckSchema.index({ monitorId: 1, createdAt: 1 });
+HardwareCheckSchema.index({ monitorId: 1, createdAt: -1 });
 
 export default mongoose.model("HardwareCheck", HardwareCheckSchema);

Server/db/mongo/MongoDB.js

@@ -91,6 +91,13 @@ class MongoDB {
 				appSettings = new AppSettings({});
 				await appSettings.save();
 			}
+			// Sync indexes
+			const models = mongoose.modelNames();
+			for (const modelName of models) {
+				const model = mongoose.model(modelName);
+				await model.syncIndexes();
+			}
+
 			logger.info({ message: "Connected to MongoDB" });
 		} catch (error) {
 			logger.error({

Server/db/mongo/modules/monitorModule.js

@@ -224,15 +224,22 @@ const getDateRange = (dateRange) => {
  * @returns {Promise<Object>} All checks and date-ranged checks
  */
 const getMonitorChecks = async (monitorId, model, dateRange, sortOrder) => {
+	const indexSpec = {
+		monitorId: 1,
+		createdAt: sortOrder, // This will be 1 or -1
+	};
+
 	const [checksAll, checksForDateRange] = await Promise.all([
-		model.find({ monitorId }).sort({ createdAt: sortOrder }),
+		model.find({ monitorId }).sort({ createdAt: sortOrder }).hint(indexSpec).lean(),
 		model
 			.find({
 				monitorId,
 				createdAt: { $gte: dateRange.start, $lte: dateRange.end },
 			})
-			.sort({ createdAt: sortOrder }),
+			.hint(indexSpec)
+			.lean(),
 	]);
+
 	return { checksAll, checksForDateRange };
 };