[
https://issues.apache.org/jira/browse/TEZ-3267?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15648950#comment-15648950
]
Hitesh Shah commented on TEZ-3267:
----------------------------------
Comments:
For tez-dag/src/main/proto/HistoryEvents.proto
tez-dag/src/main/proto/HistoryEvents.proto
- only the submitted event needs the queue name. Any reason to add it to all
others? See earlier review comments.
I believe the analyser objects need to change to parse out the queue info?
No test modifications in TestHistoryEventJsonConversion to verify publishing of
queue info?
> Allow dag to be searchable by queue name.
> -----------------------------------------
>
> Key: TEZ-3267
> URL: https://issues.apache.org/jira/browse/TEZ-3267
> Project: Apache Tez
> Issue Type: Bug
> Reporter: Harish Jaiprakash
> Assignee: Harish Jaiprakash
> Attachments: TEZ-3267.0.patch, TEZ-3267.1.patch, TEZ-3267.2.patch,
> TEZ-3267.3.patch, TEZ-3267.4.patch, TEZ-3267.5.patch, TEZ-3267.6.patch,
> TEZ-3267.7.patch
>
>
> Use submission queue info from YARN to update ATS data for dag to allow
> searches by queue name.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
