[jira] [Commented] (YARN-6804) Allow custom hostname for docker containers in native services

2017-07-13 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6804?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16086285#comment-16086285 ] Jian He commented on YARN-6804: --- - probably we can create a common util method in RegistryPat

[jira] [Commented] (YARN-6804) Allow custom hostname for docker containers in native services

2017-07-13 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6804?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16086276#comment-16086276 ] Jian He commented on YARN-6804: --- lgtm, two minor comments: - I think the check for if “LOG.i

[jira] [Commented] (YARN-6102) On failover RM can crash due to unregistered event to AsyncDispatcher

2017-07-12 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6102?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16083549#comment-16083549 ] Jian He commented on YARN-6102: --- the rmContext related looks a bit confusing. could you add c

[jira] [Commented] (YARN-2919) Potential race between renew and cancel in DelegationTokenRenwer

2017-07-06 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-2919?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16077190#comment-16077190 ] Jian He commented on YARN-2919: --- bq. assume RenewalTimerTask. run is invoked by the timer tas

[jira] [Commented] (YARN-2919) Potential race between renew and cancel in DelegationTokenRenwer

2017-07-06 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-2919?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16077028#comment-16077028 ] Jian He commented on YARN-2919: --- bq. This race is about stopping creation of the timer task

[jira] [Commented] (YARN-2919) Potential race between renew and cancel in DelegationTokenRenwer

2017-07-05 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-2919?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16075901#comment-16075901 ] Jian He commented on YARN-2919: --- got a question, [~Naganarasimha], could you elaborate what t

[jira] [Commented] (YARN-6753) Expose more ContainerImpl states from NM in ContainerStateProto

2017-06-30 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6753?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16070876#comment-16070876 ] Jian He commented on YARN-6753: --- Yep, I kinda prefer working on the existing class. Since th

[jira] [Comment Edited] (YARN-6756) ContainerRequest#executionTypeRequest causes NPE

2017-06-30 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6756?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16070675#comment-16070675 ] Jian He edited comment on YARN-6756 at 6/30/17 8:36 PM: I actually

[jira] [Comment Edited] (YARN-6756) ContainerRequest#executionTypeRequest causes NPE

2017-06-30 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6756?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16070675#comment-16070675 ] Jian He edited comment on YARN-6756 at 6/30/17 8:36 PM: I actually

[jira] [Commented] (YARN-6756) ContainerRequest#executionTypeRequest causes NPE

2017-06-30 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6756?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16070675#comment-16070675 ] Jian He commented on YARN-6756: --- I actually run into NPE when I use the builder API, if I cal

[jira] [Commented] (YARN-6756) ContainerRequest#executionTypeRequest causes NPE

2017-06-30 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6756?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16070554#comment-16070554 ] Jian He commented on YARN-6756: --- HI [~asuresh] , can you help check this ? > ContainerReques

[jira] [Updated] (YARN-6756) ContainerRequest#executionTypeRequest causes NPE

2017-06-30 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6756?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jian He updated YARN-6756: -- Priority: Critical (was: Major) > ContainerRequest#executionTypeRequest causes NPE > ---

[jira] [Created] (YARN-6756) ContainerRequest#executionTypeRequest causes NPE

2017-06-30 Thread Jian He (JIRA)
Jian He created YARN-6756: - Summary: ContainerRequest#executionTypeRequest causes NPE Key: YARN-6756 URL: https://issues.apache.org/jira/browse/YARN-6756 Project: Hadoop YARN Issue Type: Bug

[jira] [Created] (YARN-6755) MiniYarnCluster should stop all the apps when shutdown

2017-06-30 Thread Jian He (JIRA)
Jian He created YARN-6755: - Summary: MiniYarnCluster should stop all the apps when shutdown Key: YARN-6755 URL: https://issues.apache.org/jira/browse/YARN-6755 Project: Hadoop YARN Issue Type: Bug

[jira] [Commented] (YARN-6694) Add certain envs to the default yarn.nodemanager.env-whitelist

2017-06-29 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6694?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16069188#comment-16069188 ] Jian He commented on YARN-6694: --- Yeah, because certain downstream projects like hive are depe

[jira] [Commented] (YARN-6594) [API] Introduce SchedulingRequest object

2017-06-29 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6594?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16068989#comment-16068989 ] Jian He commented on YARN-6594: --- Sounds good, if SchedulingRequestBuilder has a same method f

[jira] [Commented] (YARN-6694) Add certain envs to the default yarn.nodemanager.env-whitelist

2017-06-29 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6694?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16068926#comment-16068926 ] Jian He commented on YARN-6694: --- Attached a simple patch that adds the aforementioned propert

[jira] [Updated] (YARN-6694) Add certain envs to the default yarn.nodemanager.env-whitelist

2017-06-29 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6694?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jian He updated YARN-6694: -- Attachment: YARN-6694.1.patch > Add certain envs to the default yarn.nodemanager.env-whitelist >

[jira] [Commented] (YARN-6594) [API] Introduce SchedulingRequest object

2017-06-28 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6594?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16067273#comment-16067273 ] Jian He commented on YARN-6594: --- got a question, any reason we add one more level abstraction

[jira] [Commented] (YARN-6716) Native services support for specifying component start order

2017-06-22 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16059659#comment-16059659 ] Jian He commented on YARN-6716: --- when we assign priority key to components, I think we may al

[jira] [Commented] (YARN-6413) Decouple Yarn Registry API from ZK

2017-06-21 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6413?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16058691#comment-16058691 ] Jian He commented on YARN-6413: --- Hi Ellem, sorry, I was looking at the wrong patch. some comm

[jira] [Commented] (YARN-6716) Native services support for specifying component start order

2017-06-20 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16056722#comment-16056722 ] Jian He commented on YARN-6716: --- looks good overall, minor comments - what's the difference b

[jira] [Commented] (YARN-6655) We should validate yarn native services application submission side to ensure that the hostname should be less than 63 characters

2017-06-20 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6655?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16056406#comment-16056406 ] Jian He commented on YARN-6655: --- lgtm, committing > We should validate yarn native service

[jira] [Commented] (YARN-6413) Decouple Yarn Registry API from ZK

2017-06-14 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6413?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16049782#comment-16049782 ] Jian He commented on YARN-6413: --- e.g. Several places in RegistryCli has been reformatted. Als

[jira] [Commented] (YARN-6413) Decouple Yarn Registry API from ZK

2017-06-13 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6413?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16048702#comment-16048702 ] Jian He commented on YARN-6413: --- [~ellenfkh], there are a lot of format only changes, can you

[jira] [Commented] (YARN-5648) [ATSv2 Security] Client side changes for authentication

2017-06-12 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5648?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16046844#comment-16046844 ] Jian He commented on YARN-5648: --- i see, are the UI failures related? > [ATSv2 Security] Cli

[jira] [Commented] (YARN-5648) [ATSv2 Security] Client side changes for authentication

2017-06-09 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5648?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16045132#comment-16045132 ] Jian He commented on YARN-5648: --- what is this dependency used for in the patch ? {code}

[jira] [Commented] (YARN-6638) [ATSv2 Security] Timeline reader side changes for loading auth filters and principals

2017-06-08 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6638?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16043290#comment-16043290 ] Jian He commented on YARN-6638: --- To Rohith's question, the StaticUserWebFilter was always add

[jira] [Commented] (YARN-6638) [Security] Timeline reader side changes for loading auth filters and principals

2017-06-07 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6638?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16041777#comment-16041777 ] Jian He commented on YARN-6638: --- ok, sounds good to me. > [Security] Timeline reader side ch

[jira] [Comment Edited] (YARN-6638) [Security] Timeline reader side changes for loading auth filters and principals

2017-06-07 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6638?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16041625#comment-16041625 ] Jian He edited comment on YARN-6638 at 6/7/17 9:16 PM: --- got a questio

[jira] [Commented] (YARN-6638) [Security] Timeline reader side changes for loading auth filters and principals

2017-06-07 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6638?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16041625#comment-16041625 ] Jian He commented on YARN-6638: --- got a question, how does reader read from the TimeLineReader

[jira] [Commented] (YARN-5647) [Security] Collector side changes for loading auth filters and principals

2017-06-06 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5647?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16040245#comment-16040245 ] Jian He commented on YARN-5647: --- yeah, Rohith's command worked > [Security] Collector side c

[jira] [Commented] (YARN-6511) Federation Intercepting and propagating AM-RM communications (part two: secondary subclusters added)

2017-06-06 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6511?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16040246#comment-16040246 ] Jian He commented on YARN-6511: --- lgtm, thanks > Federation Intercepting and propagating AM-R

[jira] [Created] (YARN-6694) Add certain envs to the default yarn.nodemanager.env-whitelist

2017-06-06 Thread Jian He (JIRA)
Jian He created YARN-6694: - Summary: Add certain envs to the default yarn.nodemanager.env-whitelist Key: YARN-6694 URL: https://issues.apache.org/jira/browse/YARN-6694 Project: Hadoop YARN Issue Typ

[jira] [Commented] (YARN-5647) [Security] Collector side changes for loading auth filters and principals

2017-06-06 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5647?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16039856#comment-16039856 ] Jian He commented on YARN-5647: --- could you add a log message in setTimelineFilters method to

[jira] [Updated] (YARN-6669) Kerberos support for native service AM with the service REST API

2017-06-06 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6669?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jian He updated YARN-6669: -- Attachment: YARN-6669.yarn-native-services.05.patch > Kerberos support for native service AM with the service RES

[jira] [Updated] (YARN-6669) Kerberos support for native service AM with the service REST API

2017-06-06 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6669?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jian He updated YARN-6669: -- Attachment: YARN-6669.yarn-native-services.04.patch upload a new patch that addressed the comments > Kerberos su

[jira] [Commented] (YARN-6669) Kerberos support for native service AM with the service REST API

2017-06-06 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6669?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16039714#comment-16039714 ] Jian He commented on YARN-6669: --- Thanks for the review ! bq.Is the principal definitely going

[jira] [Comment Edited] (YARN-5647) [Security] Collector side changes for loading auth filters and principals

2017-06-06 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5647?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16039447#comment-16039447 ] Jian He edited comment on YARN-5647 at 6/6/17 6:45 PM: --- I think the p

[jira] [Commented] (YARN-5647) [Security] Collector side changes for loading auth filters and principals

2017-06-06 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5647?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16039448#comment-16039448 ] Jian He commented on YARN-5647: --- Or we could validate no duplicate entries are present in ha

[jira] [Commented] (YARN-5647) [Security] Collector side changes for loading auth filters and principals

2017-06-06 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5647?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16039447#comment-16039447 ] Jian He commented on YARN-5647: --- I think the patch changed the order in which the initializer

[jira] [Commented] (YARN-6683) Invalid event: COLLECTOR_UPDATE at KILLED

2017-06-05 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6683?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16037507#comment-16037507 ] Jian He commented on YARN-6683: --- committed to trunk, thanks [~rohithsharma] ! > Invalid even

[jira] [Commented] (YARN-6683) Invalid event: COLLECTOR_UPDATE at KILLED

2017-06-05 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6683?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16037497#comment-16037497 ] Jian He commented on YARN-6683: --- lgtm, committing > Invalid event: COLLECTOR_UPDATE at KILLE

[jira] [Commented] (YARN-5647) [Security] Collector side changes for loading auth filters and principals

2017-06-05 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5647?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16037482#comment-16037482 ] Jian He commented on YARN-5647: --- bq. Intention was not to support it temporarily. Because eve

[jira] [Commented] (YARN-6511) Federation Intercepting and propagating AM-RM communications (part two: secondary subclusters added)

2017-06-05 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6511?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16037471#comment-16037471 ] Jian He commented on YARN-6511: --- i see, thanks for the explanation > Federation Intercepting

[jira] [Commented] (YARN-6511) Federation Intercepting and propagating AM-RM communications (part two: secondary subclusters added)

2017-06-05 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6511?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16037341#comment-16037341 ] Jian He commented on YARN-6511: --- thanks [~botong], not sure I'm reading it right: Looks like

[jira] [Comment Edited] (YARN-5647) [Security] Collector side changes for loading auth filters and principals

2017-06-02 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5647?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16035634#comment-16035634 ] Jian He edited comment on YARN-5647 at 6/2/17 11:42 PM: Thanks Varu

[jira] [Commented] (YARN-5647) [Security] Collector side changes for loading auth filters and principals

2017-06-02 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5647?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16035634#comment-16035634 ] Jian He commented on YARN-5647: --- Thanks - TimelineUtils#addTimelineFilters has some duplicat

[jira] [Commented] (YARN-3053) [Security] Review and implement authentication in ATS v.2

2017-06-02 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16035626#comment-16035626 ] Jian He commented on YARN-3053: --- For token renewal, (not token roll-over), the document menti

[jira] [Updated] (YARN-6669) Kerberos support for native service AM with the service REST API

2017-06-02 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6669?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jian He updated YARN-6669: -- Attachment: (was: YARN-6669.yarn-native-services.02.patch) > Kerberos support for native service AM with the

[jira] [Updated] (YARN-6669) Kerberos support for native service AM with the service REST API

2017-06-02 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6669?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jian He updated YARN-6669: -- Attachment: YARN-6669.yarn-native-services.03.patch > Kerberos support for native service AM with the service RES

[jira] [Updated] (YARN-6669) Kerberos support for native service AM with the service REST API

2017-06-02 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6669?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jian He updated YARN-6669: -- Attachment: YARN-6669.yarn-native-services.02.patch > Kerberos support for native service AM with the service RES

[jira] [Updated] (YARN-6683) Invalid event: COLLECTOR_UPDATE at KILLED

2017-06-01 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6683?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jian He updated YARN-6683: -- Description: {code} 2017-06-01 20:01:22,686 ERROR rmapp.RMAppImpl (RMAppImpl.java:handle(905)) - Can't handle th

[jira] [Created] (YARN-6683) Invalid event: COLLECTOR_UPDATE at KILLED

2017-06-01 Thread Jian He (JIRA)
Jian He created YARN-6683: - Summary: Invalid event: COLLECTOR_UPDATE at KILLED Key: YARN-6683 URL: https://issues.apache.org/jira/browse/YARN-6683 Project: Hadoop YARN Issue Type: Bug Rep

[jira] [Updated] (YARN-6669) Kerberos support for native service AM with the service REST API

2017-06-01 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6669?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jian He updated YARN-6669: -- Attachment: YARN-6669.yarn-native-services.01.patch Added a new filed in the service API spec and also removed so

[jira] [Updated] (YARN-6669) Kerberos support for native service AM with the service REST API

2017-05-30 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6669?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jian He updated YARN-6669: -- Summary: Kerberos support for native service AM with the service REST API (was: Make the native service AM read

[jira] [Created] (YARN-6669) Make the native service AM read the local keytabs

2017-05-30 Thread Jian He (JIRA)
Jian He created YARN-6669: - Summary: Make the native service AM read the local keytabs Key: YARN-6669 URL: https://issues.apache.org/jira/browse/YARN-6669 Project: Hadoop YARN Issue Type: Sub-task

[jira] [Commented] (YARN-6613) Update json validation for new native services providers

2017-05-24 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6613?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16023580#comment-16023580 ] Jian He commented on YARN-6613: --- minor comment on the patch, TestApplicationApiService can be

[jira] [Updated] (YARN-6617) Services API delete call first attempt usually fails

2017-05-22 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6617?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jian He updated YARN-6617: -- Attachment: YARN-6617-yarn-native-services.2.patch > Services API delete call first attempt usually fails > -

[jira] [Comment Edited] (YARN-6613) Update json validation for new native services providers

2017-05-22 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6613?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16020188#comment-16020188 ] Jian He edited comment on YARN-6613 at 5/22/17 9:00 PM: Thanks Bill

[jira] [Commented] (YARN-6613) Update json validation for new native services providers

2017-05-22 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6613?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16020188#comment-16020188 ] Jian He commented on YARN-6613: --- Thanks Billie, patch looks good overall, some comments I had

[jira] [Commented] (YARN-6627) Use deployed webapp folder to launch new YARN UI

2017-05-19 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6627?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16018190#comment-16018190 ] Jian He commented on YARN-6627: --- [~sunilg], patch looks good overall, some comments about the

[jira] [Updated] (YARN-6627) Use deployed webapp folder to launch new YARN UI

2017-05-19 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6627?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jian He updated YARN-6627: -- Attachment: YARN-6627.0003.patch > Use deployed webapp folder to launch new YARN UI > ---

[jira] [Commented] (YARN-6602) Impersonation does not work if standby RM is contacted first

2017-05-19 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6602?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16018094#comment-16018094 ] Jian He commented on YARN-6602: --- Patch looks good to me too. Thanks ! > Impersonation does n

[jira] [Updated] (YARN-6617) Services API delete call first attempt usually fails

2017-05-17 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6617?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jian He updated YARN-6617: -- Attachment: YARN-6617-yarn-native-services.1.patch Made the actionStop wait for a few seconds and then do a force

[jira] [Assigned] (YARN-6617) Services API delete call first attempt usually fails

2017-05-17 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6617?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jian He reassigned YARN-6617: - Assignee: Jian He > Services API delete call first attempt usually fails > ---

[jira] [Commented] (YARN-6160) Create an agent-less docker-less provider in the native services framework

2017-05-17 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6160?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16014387#comment-16014387 ] Jian He commented on YARN-6160: --- sounds good, I'll commit the current patch > Create an agen

[jira] [Commented] (YARN-6160) Create an agent-less docker-less provider in the native services framework

2017-05-16 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6160?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16013524#comment-16013524 ] Jian He commented on YARN-6160: --- lgtm, for the findbugs warnings, what is the doClientInstall

[jira] [Commented] (YARN-6160) Create an agent-less docker-less provider in the native services framework

2017-05-16 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6160?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16013070#comment-16013070 ] Jian He commented on YARN-6160: --- looks good over all, minor comments: - remove unused variabl

[jira] [Commented] (YARN-6306) NMClient API change for container upgrade

2017-05-15 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6306?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16011431#comment-16011431 ] Jian He commented on YARN-6306: --- +1, thanks Arun! > NMClient API change for container upgrad

[jira] [Commented] (YARN-5287) LinuxContainerExecutor fails to set proper permission

2017-05-15 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5287?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16011057#comment-16011057 ] Jian He commented on YARN-5287: --- [~Naganarasimha], [~haibo.chen], saw a bunch of comments on

[jira] [Commented] (YARN-6306) NMClient API change for container upgrade

2017-05-13 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6306?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16009502#comment-16009502 ] Jian He commented on YARN-6306: --- Hi Arun, looks good to me, minor things: - NMClient#getNodeI

[jira] [Comment Edited] (YARN-6591) Embed NodeId as part of ContainerId ?

2017-05-12 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6591?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16008719#comment-16008719 ] Jian He edited comment on YARN-6591 at 5/12/17 9:41 PM: Yeah, it wo

[jira] [Commented] (YARN-6591) Embed NodeId as part of ContainerId ?

2017-05-12 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6591?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16008719#comment-16008719 ] Jian He commented on YARN-6591: --- Yeah, it would need be an additional field. - We may append

[jira] [Commented] (YARN-6306) NMClient API change for container upgrade

2017-05-12 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6306?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16008510#comment-16008510 ] Jian He commented on YARN-6306: --- Ok, filed YARN-6591 for the thoughts. > NMClient API change

[jira] [Created] (YARN-6591) Embed NodeId as part of ContainerId ?

2017-05-12 Thread Jian He (JIRA)
Jian He created YARN-6591: - Summary: Embed NodeId as part of ContainerId ? Key: YARN-6591 URL: https://issues.apache.org/jira/browse/YARN-6591 Project: Hadoop YARN Issue Type: Bug Reporte

[jira] [Commented] (YARN-6306) NMClient API change for container upgrade

2017-05-12 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6306?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16008429#comment-16008429 ] Jian He commented on YARN-6306: --- few more comments: The NMClient#getNodeId API may cause conf

[jira] [Updated] (YARN-6588) Add native-service AM log4j properties in classpath

2017-05-11 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6588?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jian He updated YARN-6588: -- Attachment: YARN-6588.yarn-native-services.01.patch > Add native-service AM log4j properties in classpath > -

[jira] [Created] (YARN-6588) Add native-service AM log4j properties in classpath

2017-05-11 Thread Jian He (JIRA)
Jian He created YARN-6588: - Summary: Add native-service AM log4j properties in classpath Key: YARN-6588 URL: https://issues.apache.org/jira/browse/YARN-6588 Project: Hadoop YARN Issue Type: Sub-task

[jira] [Commented] (YARN-6533) Race condition in writing service record to registry in yarn native services

2017-05-10 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6533?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16005346#comment-16005346 ] Jian He commented on YARN-6533: --- yeah, sounds good to me, we can leave it unencoded. Else the

[jira] [Comment Edited] (YARN-6533) Race condition in writing service record to registry in yarn native services

2017-05-09 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6533?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16004045#comment-16004045 ] Jian He edited comment on YARN-6533 at 5/10/17 6:08 AM: In SliderAp

[jira] [Commented] (YARN-6533) Race condition in writing service record to registry in yarn native services

2017-05-09 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6533?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16004045#comment-16004045 ] Jian He commented on YARN-6533: --- In SliderAppMaster Below code also didn't replace '_' with '

[jira] [Commented] (YARN-6533) Race condition in writing service record to registry in yarn native services

2017-05-09 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6533?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16003496#comment-16003496 ] Jian He commented on YARN-6533: --- Actually, there's an issue, previously, componentInstanceSta

[jira] [Commented] (YARN-6533) Race condition in writing service record to registry in yarn native services

2017-05-09 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6533?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16003474#comment-16003474 ] Jian He commented on YARN-6533: --- +1, committing > Race condition in writing service record

[jira] [Updated] (YARN-6545) Followup fix for YARN-6405

2017-05-08 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6545?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jian He updated YARN-6545: -- Attachment: YARN-6545.yarn-native-services.07.patch > Followup fix for YARN-6405 > -- > >

[jira] [Updated] (YARN-6545) Followup fix for YARN-6405

2017-05-08 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6545?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jian He updated YARN-6545: -- Attachment: YARN-6545.yarn-native-services.06.patch > Followup fix for YARN-6405 > -- > >

[jira] [Updated] (YARN-6545) Followup fix for YARN-6405

2017-05-08 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6545?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jian He updated YARN-6545: -- Attachment: (was: YARN-6545.yarn-native-services.06.patch) > Followup fix for YARN-6405 > ---

[jira] [Updated] (YARN-6545) Followup fix for YARN-6405

2017-05-08 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6545?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jian He updated YARN-6545: -- Attachment: YARN-6545.yarn-native-services.06.patch > Followup fix for YARN-6405 > -- > >

[jira] [Commented] (YARN-6545) Followup fix for YARN-6405

2017-05-08 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6545?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16001222#comment-16001222 ] Jian He commented on YARN-6545: --- [~billie.rinaldi], I updated on top of the last patch in a f

[jira] [Updated] (YARN-6545) Followup fix for YARN-6405

2017-05-08 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6545?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jian He updated YARN-6545: -- Attachment: YARN-6545.yarn-native-services.05.patch > Followup fix for YARN-6405 > -- > >

[jira] [Commented] (YARN-6306) NMClient API change for container upgrade

2017-05-06 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6306?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15999670#comment-15999670 ] Jian He commented on YARN-6306: --- Hi [~asuresh], I did a quick scan, - wonder why not add the

[jira] [Commented] (YARN-6544) Add Null check RegistryDNS service while parsing registry records

2017-05-02 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6544?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15993770#comment-15993770 ] Jian He commented on YARN-6544: --- [~karams], we may add a warning log in the else case, if the

[jira] [Updated] (YARN-6545) Followup fix for YARN-6405

2017-05-02 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6545?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jian He updated YARN-6545: -- Attachment: (was: YARN-6545.yarn-native-services.01.patch) > Followup fix for YARN-6405 > ---

[jira] [Updated] (YARN-6545) Followup fix for YARN-6405

2017-05-02 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6545?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jian He updated YARN-6545: -- Attachment: YARN-6545.yarn-native-services.01.patch > Followup fix for YARN-6405 > -- > >

[jira] [Updated] (YARN-6545) Followup fix for YARN-6405

2017-05-02 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6545?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jian He updated YARN-6545: -- Attachment: YARN-6545.yarn-native-services.01.patch > Followup fix for YARN-6405 > -- > >

[jira] [Updated] (YARN-6545) Followup fix for YARN-6405

2017-05-02 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6545?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jian He updated YARN-6545: -- Issue Type: Sub-task (was: Bug) Parent: YARN-5079 > Followup fix for YARN-6405 > ---

[jira] [Created] (YARN-6545) Followup fix for YARN-6405

2017-05-02 Thread Jian He (JIRA)
Jian He created YARN-6545: - Summary: Followup fix for YARN-6405 Key: YARN-6545 URL: https://issues.apache.org/jira/browse/YARN-6545 Project: Hadoop YARN Issue Type: Bug Reporter: Jian He

[jira] [Commented] (YARN-6419) Support to launch native-service deployment from new YARN UI

2017-05-02 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6419?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15993180#comment-15993180 ] Jian He commented on YARN-6419: --- lgtm, thanks > Support to launch native-service deployment

[jira] [Commented] (YARN-1197) Support changing resources of an allocated container

2017-04-28 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-1197?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15989772#comment-15989772 ] Jian He commented on YARN-1197: --- [~mingma], I don't think fair scheduler support this as of n

[jira] [Comment Edited] (YARN-6405) Improve configuring services through REST API

2017-04-23 Thread Jian He (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6405?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15980693#comment-15980693 ] Jian He edited comment on YARN-6405 at 4/24/17 3:45 AM: Thanks for

<    2   3   4   5   6   7   8   9   10   11   >