TaoLv commented on a change in pull request #11251: [WIP] Graph partitioner and subgraph op URL: https://github.com/apache/incubator-mxnet/pull/11251#discussion_r195468081
########## File path: include/mxnet/op_attr_types.h ########## @@ -98,7 +98,12 @@ enum class ExecType { * In current implementation, copy operator is specially handled by executor. * This flag is used for special case treatment and future extension of different copy ops. */ - kCrossDeviceCopy + kCrossDeviceCopy, + /*! + * A subgraph execution should happen in the main thread, instead of + * in the execution engine. Review comment: operators inside a subgraph are still executed in the execution engine, right? ---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services