Github user zentol commented on a diff in the pull request:
https://github.com/apache/flink/pull/4636#discussion_r136817884
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/util/EnvironmentInformation.java
---
@@ -284,7 +284,16 @@ public static void logEnvironmentInfo(Logger log,
String componentName, String[]
log.info(" JVM: " + jvmVersion);
log.info(" Maximum heap size: " + maxHeapMegabytes + "
MiBytes");
log.info(" JAVA_HOME: " + (javaHome == null ? "(not
set)" : javaHome));
- log.info(" Hadoop version: " +
VersionInfo.getVersion());
+
+ try {
+ Class.forName(
+ "org.apache.hadoop.util.VersionInfo",
+ false,
+
EnvironmentInformation.class.getClassLoader());
+ log.info(" Hadoop version: " +
VersionInfo.getVersion());
--- End diff --
Did you intend to directly call `VersionInfo`, or should we maybe do this
with reflection instead?
---
