maheshrajus commented on code in PR #474:
URL: https://github.com/apache/tez/pull/474#discussion_r3381594187
##########
tez-api/src/main/java/org/apache/tez/dag/api/client/TimelineReaderFactory.java:
##########
@@ -221,17 +221,17 @@ public
TokenAuthenticatedURLConnectionFactory(ConnectionConfigurator connConfigu
}
@Override
- public HttpURLConnection getHttpURLConnection(URL url) throws
IOException {
+ public HttpURLConnection getConnection(URL url) throws IOException {
try {
- AuthenticatedURL authenticatedURL=
ReflectionUtils.createClazzInstance(
+ Object authenticatedURL = ReflectionUtils.createClazzInstance(
Review Comment:
Fixed
##########
tez-api/src/main/java/org/apache/tez/dag/api/client/TimelineReaderFactory.java:
##########
@@ -221,17 +221,17 @@ public
TokenAuthenticatedURLConnectionFactory(ConnectionConfigurator connConfigu
}
@Override
- public HttpURLConnection getHttpURLConnection(URL url) throws
IOException {
+ public HttpURLConnection getConnection(URL url) throws IOException {
try {
- AuthenticatedURL authenticatedURL=
ReflectionUtils.createClazzInstance(
+ Object authenticatedURL = ReflectionUtils.createClazzInstance(
DELEGATION_TOKEN_AUTHENTICATED_URL_CLAZZ_NAME, new Class[] {
delegationTokenAuthenticatorClazz,
ConnectionConfigurator.class
}, new Object[] {
authenticator,
connConfigurator
});
- return ReflectionUtils.invokeMethod(authenticatedURL,
+ return (HttpURLConnection)
ReflectionUtils.invokeMethod(authenticatedURL,
Review Comment:
Fixed
--
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]
