seojangho commented on a change in pull request #48: [NEMO-72] Instance-based 
Encoder/Decoder interface
URL: https://github.com/apache/incubator-nemo/pull/48#discussion_r196273838
 
 

 ##########
 File path: 
compiler/frontend/beam/src/main/java/edu/snu/nemo/compiler/frontend/beam/NemoPipelineVisitor.java
 ##########
 @@ -187,35 +196,40 @@ public void visitPrimitiveTransform(final 
TransformHierarchy.Node beamNode) {
 
   /**
    * Connect side inputs to the vertex.
-   * @param builder the DAG builder to add the vertex to.
-   * @param sideInputs side inputs.
+   *
+   * @param builder        the DAG builder to add the vertex to.
+   * @param sideInputs     side inputs.
    * @param pValueToVertex PValue to Vertex map.
-   * @param pValueToCoder  PValue to Coder map.
-   * @param irVertex wrapper for a user operation in the IR. (Where the side 
input is headed to)
+   * @param pValueToCoder  PValue to Encoder map.
 
 Review comment:
   `PValue to Encoder/Decoder map.`

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to