YARN-5271. ATS client doesn't work with Jersey 2 on the classpath. Contributed by Weiwei Yang.
(cherry picked from commit09520cb439
) (cherry picked from commitc5a3ec493b
)
This commit is contained in:
parent
416f6f3605
commit
20c90e13b8
|
@ -174,11 +174,24 @@ public class YarnClientImpl extends YarnClient {
|
|||
|
||||
if (conf.getBoolean(YarnConfiguration.TIMELINE_SERVICE_ENABLED,
|
||||
YarnConfiguration.DEFAULT_TIMELINE_SERVICE_ENABLED)) {
|
||||
timelineServiceEnabled = true;
|
||||
timelineClient = createTimelineClient();
|
||||
timelineClient.init(conf);
|
||||
timelineDTRenewer = getTimelineDelegationTokenRenewer(conf);
|
||||
timelineService = TimelineUtils.buildTimelineTokenService(conf);
|
||||
try {
|
||||
timelineServiceEnabled = true;
|
||||
timelineClient = createTimelineClient();
|
||||
timelineClient.init(conf);
|
||||
timelineDTRenewer = getTimelineDelegationTokenRenewer(conf);
|
||||
timelineService = TimelineUtils.buildTimelineTokenService(conf);
|
||||
} catch (NoClassDefFoundError error) {
|
||||
// When attempt to initiate the timeline client with
|
||||
// different set of dependencies, it may fail with
|
||||
// NoClassDefFoundError. When some of them are not compatible
|
||||
// with timeline server. This is not necessarily a fatal error
|
||||
// to the client.
|
||||
LOG.warn("Timeline client could not be initialized "
|
||||
+ "because dependency missing or incompatible,"
|
||||
+ " disabling timeline client.",
|
||||
error);
|
||||
timelineServiceEnabled = false;
|
||||
}
|
||||
}
|
||||
|
||||
timelineServiceBestEffort = conf.getBoolean(
|
||||
|
|
|
@ -18,6 +18,7 @@
|
|||
|
||||
package org.apache.hadoop.yarn.client.api.impl;
|
||||
|
||||
import static org.junit.Assert.assertFalse;
|
||||
import static org.mockito.Matchers.any;
|
||||
import static org.mockito.Mockito.mock;
|
||||
import static org.mockito.Mockito.never;
|
||||
|
@ -155,6 +156,26 @@ public class TestYarnClient {
|
|||
rm.stop();
|
||||
}
|
||||
|
||||
@Test
|
||||
public void testTimelineClientInitFailure() throws Exception{
|
||||
Configuration conf = new Configuration();
|
||||
conf.setBoolean(YarnConfiguration.TIMELINE_SERVICE_ENABLED, true);
|
||||
YarnClient client = YarnClient.createYarnClient();
|
||||
if(client instanceof YarnClientImpl) {
|
||||
YarnClientImpl impl = (YarnClientImpl) client;
|
||||
YarnClientImpl spyClient = spy(impl);
|
||||
when(spyClient.createTimelineClient()).thenThrow(
|
||||
new NoClassDefFoundError(
|
||||
"Mock a failure when init timeline instance"));
|
||||
spyClient.init(conf);
|
||||
spyClient.start();
|
||||
assertFalse("Timeline client should be disabled when"
|
||||
+ "it is failed to init",
|
||||
spyClient.timelineServiceEnabled);
|
||||
spyClient.stop();
|
||||
}
|
||||
}
|
||||
|
||||
@SuppressWarnings("deprecation")
|
||||
@Test (timeout = 30000)
|
||||
public void testSubmitApplication() throws Exception {
|
||||
|
|
Loading…
Reference in New Issue