Don't yield in tight loop when context process hasn't started (#5011)

aws · Oct 28, 2024 · 987e2ae · 987e2ae
1 parent a81caf6
commit 987e2ae
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/.changes/next-release/bugfix-b18a6b61-e1e9-42dd-952d-7c32ea7a3a19.json b/.changes/next-release/bugfix-b18a6b61-e1e9-42dd-952d-7c32ea7a3a19.json
@@ -0,0 +1,4 @@
+{
+  "type" : "bugfix",
+  "description" : "Fix pointless busy loop in Amazon Q wasting CPU cycles (#5000)"
+}
diff --git a/...ty/src/software/aws/toolkits/jetbrains/services/amazonq/project/ProjectContextProvider.kt b/...ty/src/software/aws/toolkits/jetbrains/services/amazonq/project/ProjectContextProvider.kt
@@ -21,7 +21,6 @@ import kotlinx.coroutines.delay
 import kotlinx.coroutines.launch
 import kotlinx.coroutines.runBlocking
 import kotlinx.coroutines.withTimeout
-import kotlinx.coroutines.yield
 import software.aws.toolkits.core.utils.debug
 import software.aws.toolkits.core.utils.error
 import software.aws.toolkits.core.utils.getLogger
@@ -48,14 +47,15 @@ class ProjectContextProvider(val project: Project, private val encoderServer: En
                 return@launch
             }
 
+            // TODO: need better solution for this
+            @Suppress("LoopWithTooManyJumpStatements")
             while (true) {
                 if (encoderServer.isNodeProcessRunning()) {
-                    // TODO: need better solution for this
                     delay(10000)
                     initAndIndex()
                     break
                 } else {
-                    yield()
+                    delay(10000)
                 }
             }
         }