gsadmin/AutopilotMonitor
1
0
Fork 0
mirror of https://github.com/okieselbach/AutopilotMonitor.git synced 2026-10-24 12:42:38 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(backend): bound QuickSearch typeahead scan (recent window + row cap)

Browse Source 
QuickSearchSessionsAsync walked the full SessionsIndex partition (or the
whole table for global scope) on every typeahead request, since
exact-over-fuzzy ordering forbids an early break on fuzzy-only/no-match
queries. Cost grew with the index.

Bound the scan without a new index:
- recent-window RowKey bound (RowKey lt cutoff, 90d) via the existing
  ComputeCutoffRowKeyPrefix; applies to tenant and global scope
- hard scanned-row cap (5000) as backstop for the no-early-break case

Match/ordering logic unchanged.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
Oliver Kieselbach
2026-06-25 11:07:29 +02:00
parent 27ef527e80
commit 12dabbeab9
1 changed files with 21 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/Backend/AutopilotMonitor.Functions/Services/TableStorageService.AgentApi.cs
+21 -4
View File
@@ -307,6 +307,12 @@ namespace AutopilotMonitor.Functions.Services
// ===== SEARCH METHODS =====
/// <summary>Typeahead only inspects sessions started within this many days (recent-window scan bound).</summary>
private const int QuickSearchWindowDays = 90;
/// <summary>Hard upper bound on index rows a single typeahead request may scan (cost backstop).</summary>
private const int QuickSearchMaxScannedRows = 5000;
/// <summary>
/// Lightweight typeahead search: matches SessionId, SerialNumber, or DeviceName.
/// Supports exact contains match (priority) and fuzzy match (edit distance &lt;= 2).
@@ -319,13 +325,24 @@ namespace AutopilotMonitor.Functions.Services
var fuzzyResults = new List<QuickSearchResult>();
var q = query.Trim();
// Tenant-scoped or cross-tenant scan
string? filter = !string.IsNullOrEmpty(tenantId)
? $"PartitionKey eq '{tenantId}'"
: null;
// Bound the scan so a single typeahead request cannot walk an ever-growing index:
// 1. Recent-window bound — RowKey is the inverted-tick prefix (newest-first), so
// `RowKey lt '{cutoff}'` keeps the scan to sessions started within the window.
// Typeahead targets recent enrollments; older sessions are out of scope.
// 2. Hard scanned-row cap — backstop for the fuzzy-only / no-match case, which
// otherwise reads every remaining row in the partition (or the whole table for
// global scope) because exact-over-fuzzy ordering forbids an early break.
var windowBound = $"RowKey lt '{ComputeCutoffRowKeyPrefix(QuickSearchWindowDays)}'";
string filter = !string.IsNullOrEmpty(tenantId)
? $"PartitionKey eq '{tenantId}' and {windowBound}"
: windowBound;
var scannedRows = 0;
await foreach (var entity in indexTableClient.QueryAsync<TableEntity>(filter: filter))
{
if (++scannedRows > QuickSearchMaxScannedRows)
break;
var sessionId = entity.GetString("SessionId") ?? ExtractSessionIdFromIndexRowKey(entity.RowKey);
var serial = entity.GetString("SerialNumber") ?? string.Empty;
var deviceName = entity.GetString("DeviceName") ?? string.Empty;