perf: Skip a write to storage when nothing is actually changing

This covers both `set` and `merge`
Expensify · Jul 27, 2021 · 908ec0a · 908ec0a
1 parent 9549e40
commit 908ec0a
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/lib/Onyx.js b/lib/Onyx.js
@@ -527,6 +527,11 @@ function evictStorageAndRetry(error, ionMethod, ...args) {
  * @returns {Promise}
  */
 function set(key, val) {
+    // Skip writing to storage if the value hasn't changed
+    if (cache.hasCacheForKey(key) && _.isEqual(val, cache.getValue(key))) {
+        return Promise.resolve();
+    }
+
     // Adds the key to cache when it's not available
     cache.set(key, val);