@azure/app-configuration-importer 3.0.0-preview → 3.0.1-preview

package/dist/index.js

@@ -19097,6 +19097,7 @@
          */
         async GetConfigurationChanges(configSettingsSource, strict = false, importMode = exports.ImportMode.IgnoreMatch, customHeadersOption) {
             var _a, e_1, _b, _c;
+            var _d, _e;
             this.validateImportMode(importMode);
             // Generate correlationRequestId for operations in the same activity
             if (!customHeadersOption) {
@@ -19117,33 +19118,33 @@
             }
             const configSettings = configSettingsResult;
             const configurationChanges = [];
-            const configurationSettingToAdd = [];
-            const srcKeyLabelLookUp = {};
-            configSettings.forEach((config) => {
-                if (!srcKeyLabelLookUp[config.key]) {
-                    srcKeyLabelLookUp[config.key] = {};
-                }
-                srcKeyLabelLookUp[config.key][config.label || ""] = true;
-            });
-            configurationSettingToAdd.push(...configSettings);
+            // Build O(1) lookup structures keyed by "key\0label" composite.
+            const srcMap = new Map();
+            const toAddKeys = new Set();
+            for (const config of configSettings) {
+                const composite = `${config.key}\u0000${(_d = config.label) !== null && _d !== void 0 ? _d : ""}`;
+                srcMap.set(composite, config);
+                toAddKeys.add(composite);
+            }
             try {
-                for (var _d = true, _e = __asyncValues(this.configurationClient.listConfigurationSettings(Object.assign(Object.assign({}, configSettingsSource.FilterOptions), customHeadersOption))), _f; _f = await _e.next(), _a = _f.done, !_a;) {
-                    _c = _f.value;
-                    _d = false;
+                // Stream target settings so we don't hold the entire remote store in memory.
+                for (var _f = true, _g = __asyncValues(this.configurationClient.listConfigurationSettings(Object.assign(Object.assign({}, configSettingsSource.FilterOptions), customHeadersOption))), _h; _h = await _g.next(), _a = _h.done, !_a;) {
+                    _c = _h.value;
+                    _f = false;
                     try {
                         const existing = _c;
-                        const isKeyLabelPresent = srcKeyLabelLookUp[existing.key] && srcKeyLabelLookUp[existing.key][existing.label || ""];
-                        if (strict && !isKeyLabelPresent) {
+                        const composite = `${existing.key}\u0000${(_e = existing.label) !== null && _e !== void 0 ? _e : ""}`;
+                        const incoming = srcMap.get(composite);
+                        if (strict && !incoming) {
                             configurationChanges.push({
                                 changeType: exports.ChangeType.Delete,
                                 currentValue: existing,
                                 newValue: null
                             });
                         }
-                        const incoming = configSettings.find(configSetting => configSetting.key == existing.key && configSetting.label === existing.label);
                         if (incoming) {
                             // Remove from add list since it already exists
-                            configurationSettingToAdd.splice(configurationSettingToAdd.indexOf(incoming), 1);
+                            toAddKeys.delete(composite);
                             if (!isConfigSettingEqual(incoming, existing)) {
                                 configurationChanges.push({
                                     changeType: exports.ChangeType.Update,
@@ -19161,22 +19162,22 @@
                         }
                     }
                     finally {
-                        _d = true;
+                        _f = true;
                     }
                 }
             }
             catch (e_1_1) { e_1 = { error: e_1_1 }; }
             finally {
                 try {
-                    if (!_d && !_a && (_b = _e.return)) await _b.call(_e);
+                    if (!_f && !_a && (_b = _g.return)) await _b.call(_g);
                 }
                 finally { if (e_1) throw e_1.error; }
             }
-            for (const setting of configurationSettingToAdd) {
+            for (const composite of toAddKeys) {
                 configurationChanges.push({
                     changeType: exports.ChangeType.Create,
                     currentValue: null,
-                    newValue: setting
+                    newValue: srcMap.get(composite)
                 });
             }
             return configurationChanges;