Add TODO to remove classes after migration to configurable attributes

Reviewed By: bobyangyf

fbshipit-source-id: 33563d7534

src/com/facebook/buck/parser/DefaultParser.java

@@ -45,6 +45,7 @@
  * <p>Computed targets are cached but are automatically invalidated if Watchman reports any
  * filesystem changes that may affect computed results.
  */
+// TODO: remove after migration to configurable attributes
 class DefaultParser extends AbstractParser {
 
   private static final Logger LOG = Logger.get(Parser.class);

src/com/facebook/buck/parser/LegacyPerBuildStateFactory.java

@@ -37,6 +37,7 @@
  * This factory creates {@link PerBuildStateFactory} that uses simple pipeline that doesn't support
  * configurable attributes.
  */
+// TODO: remove after migration to configurable attributes
 class LegacyPerBuildStateFactory extends PerBuildStateFactory {
   private final TypeCoercerFactory typeCoercerFactory;
   private final ConstructorArgMarshaller marshaller;

src/com/facebook/buck/parser/TargetNodeParsePipeline.java

@@ -52,6 +52,7 @@
  * duplicated (the **JobsCache variables) are used to make sure we don't schedule the same work more
  * than once), however it's possible for multiple read-only commands to duplicate work.
  */
+// TODO: remove after migration to configurable attributes
 @ThreadSafe
 public class TargetNodeParsePipeline
     extends ConvertingPipeline<Map<String, Object>, TargetNode<?>> {