summon: revert dvc.api._invoke_mehtod() considerations

It's now thread-safe, but the rest is still relevant.
iterative · Jan 13, 2020 · d0de9d0 · d0de9d0
1 parent 4a3dd63
commit d0de9d0
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/dvc/api.py b/dvc/api.py
@@ -187,6 +187,10 @@ def _get_object_desc(name, path):
 
 @wrap_with(threading.Lock())
 def _invoke_method(call, args, path):
+    # XXX: Some issues with this approach:
+    #   * Import will pollute sys.modules
+    #   * sys.path manipulation is "theoretically" not needed,
+    #     but tests are failing for an unknown reason.
     cwd = os.getcwd()
 
     try: