Fix CPU metrics mixed for different Azure SQL databases on same server (#680)

GetServerNameForStorage() used only the server hostname to generate
storage keys, so multiple Azure SQL databases on the same logical
server (e.g. myserver.database.windows.net) produced identical
server_ids and their metrics were conflated.

Include DatabaseName in the storage key when set, following the
existing pattern used for ReadOnlyIntent connections.

Fixes #677

Co-authored-by: Erik Darling <2136037+erikdarlingdata@users.noreply.github.com>
Co-authored-by: thithanhtuyennguyen <tuyen.nguyen@wwinc.com>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: 4 people

Lite/Services/RemoteCollectorService.cs

@@ -682,12 +682,18 @@ protected static long GenerateCollectionId()
 
     /// <summary>
     /// Gets the server name used for DuckDB storage and hashing.
+    /// Appends the database name for Azure SQL Database connections so that
+    /// different databases on the same logical server get distinct server_ids.
     /// Appends ":RO" for ReadOnlyIntent connections so they get a
     /// different server_id than read-write connections to the same host.
     /// </summary>
     internal static string GetServerNameForStorage(ServerConnection server)
     {
-        return server.ReadOnlyIntent ? server.ServerName + ":RO" : server.ServerName;
+        var name = string.IsNullOrWhiteSpace(server.DatabaseName)
+            ? server.ServerName
+            : server.ServerName + ":" + server.DatabaseName;
+
+        return server.ReadOnlyIntent ? name + ":RO" : name;
     }
 
     /// <summary>