Apply suggestions from code review

[skip ci] Co-authored-by: Alessio Placitelli <alessio.placitelli@gmail.com>
mozilla · Jul 22, 2020 · e9e7154 · e9e7154
1 parent 8f670ce
commit e9e7154
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/glean-core/android/src/main/java/mozilla/telemetry/glean/Glean.kt b/glean-core/android/src/main/java/mozilla/telemetry/glean/Glean.kt
@@ -296,7 +296,7 @@ open class GleanInternalAPI internal constructor () {
             Pass the correct state into `Glean.initialize()`.
             See documentation at https://mozilla.github.io/glean/book/user/general-api.html#initializing-the-glean-sdk
             """.trimIndent()
-            Log.w(LOG_TAG, msg)
+            Log.e(LOG_TAG, msg)
             return
         }
         // Changing upload enabled always happens asynchronous.

diff --git a/glean-core/csharp/Glean/Glean.cs b/glean-core/csharp/Glean/Glean.cs
@@ -266,7 +266,7 @@ public void SetUploadEnabled(bool enabled)
                 string msg = "Changing upload enabled before Glean is initialized is not supported.\n" +
                     "Pass the correct state into `Glean.initialize()`.\n" +
                     "See documentation at https://mozilla.github.io/glean/book/user/general-api.html#initializing-the-glean-sdk";
-                Log.Warning(msg);
+                Log.Error(msg);
 
                 return;
             }

diff --git a/glean-core/python/glean/glean.py b/glean-core/python/glean/glean.py
@@ -312,7 +312,7 @@ def set_upload_enabled(cls, enabled: bool) -> None:
                   See documentation at
                   https://mozilla.github.io/glean/book/user/general-api.html#initializing-the-glean-sdk
                   """
-            log.warning(inspect.cleandoc(msg))
+            log.error(inspect.cleandoc(msg))
             return
 
         # Changing upload enabled always happens asynchronous.