abstractdog commented on a change in pull request #166:
URL: https://github.com/apache/tez/pull/166#discussion_r775144161
##########
File path: tez-common/src/main/java/org/apache/tez/dag/records/DAGIDAware.java
##########
@@ -0,0 +1,11 @@
+package org.apache.tez.dag.records;
+
+import org.apache.hadoop.yarn.api.records.ApplicationId;
+
+public interface DAGIDAware {
+ TezDAGID getDAGId();
Review comment:
even if it looks strange (too many capitals), this might be changed to
```
getDAGID();
```
to stay consistent with others, so it should suggest that a TezID Object is
returned here which is "ID", not an integer, which is id
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
