Add a TODO

Change-Id: I92bce9649f9cb91474a835b2ea937b4a740e2bfe

src/main/java/com/google/devtools/build/lib/remote/RemoteModule.java

@@ -548,6 +548,13 @@ public void afterAnalysis(
     }
   }
 
+  // This is a short-term fix for top-level outputs that are symlinks. Unfortunately, we cannot
+  // reliably tell after analysis whether actions will create symlinks (the RE protocol allows any
+  // action to generate and return symlinks), but at least we can handle basic C++ rules with this
+  // change.
+  // TODO(ulfjack): I think we should separate downloading files from action execution. That would
+  // also resolve issues around action invalidation - we currently invalidate actions to trigger
+  // downloads of top-level outputs when the top-level targets change.
   private static void fetchSymlinkDependenciesRecursively(
       ActionGraph actionGraph, Set<ActionInput> builder, List<Artifact> inputs) {
     for (Artifact input : inputs) {