You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
When dfs.federation.router.federation.rename.option is set to DISTCP and hdfs.fedbalance.procedure.scheduler.journal.uri is not set, DFSRouter fails to launch.
2021-09-08 15:39:11,818 ERROR org.apache.hadoop.hdfs.server.federation.router.DFSRouter: Failed to start router
java.lang.NullPointerException
at java.base/java.net.URI$Parser.parse(URI.java:3104)
at java.base/java.net.URI.<init>(URI.java:600)
at org.apache.hadoop.hdfs.server.federation.router.RouterRpcServer.initRouterFedRename(RouterRpcServer.java:444)
at org.apache.hadoop.hdfs.server.federation.router.RouterRpcServer.<init>(RouterRpcServer.java:419)
at org.apache.hadoop.hdfs.server.federation.router.Router.createRpcServer(Router.java:391)
at org.apache.hadoop.hdfs.server.federation.router.Router.serviceInit(Router.java:188)
at org.apache.hadoop.service.AbstractService.init(AbstractService.java:164)
at org.apache.hadoop.hdfs.server.federation.router.DFSRouter.main(DFSRouter.java:69)
hdfs.fedbalance.procedure.scheduler.journal.uri is hdfs://localhost:8020/tmp/procedure by default, however, the default value is not used in DFSRouter.
How was this patch tested?
Local dev testing. After applying fix for HDFS-16219, we can reproduce above error. And the fix provides default value to hdfs.fedbalance.procedure.scheduler.journal.uri.
For code changes:
Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ trunk Compile Tests _
+1 💚
mvninstall
33m 38s
trunk passed
+1 💚
compile
0m 41s
trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚
compile
0m 35s
trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+1 💚
checkstyle
0m 23s
trunk passed
+1 💚
mvnsite
0m 42s
trunk passed
+1 💚
javadoc
0m 38s
trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚
javadoc
0m 53s
trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+1 💚
spotbugs
1m 15s
trunk passed
+1 💚
shadedclient
17m 1s
branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+1 💚
mvninstall
0m 34s
the patch passed
+1 💚
compile
0m 35s
the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚
javac
0m 35s
the patch passed
+1 💚
compile
0m 30s
the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+1 💚
javac
0m 30s
the patch passed
+1 💚
blanks
0m 0s
The patch has no blanks issues.
+1 💚
checkstyle
0m 15s
the patch passed
+1 💚
mvnsite
0m 32s
the patch passed
+1 💚
javadoc
0m 32s
the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚
javadoc
0m 47s
the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+1 💚
spotbugs
1m 19s
the patch passed
+1 💚
shadedclient
16m 51s
patch has no errors when building and testing our client artifacts.
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ trunk Compile Tests _
+1 💚
mvninstall
33m 27s
trunk passed
+1 💚
compile
0m 42s
trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚
compile
0m 34s
trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+1 💚
checkstyle
0m 23s
trunk passed
+1 💚
mvnsite
0m 40s
trunk passed
+1 💚
javadoc
0m 37s
trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚
javadoc
0m 52s
trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+1 💚
spotbugs
1m 14s
trunk passed
+1 💚
shadedclient
17m 6s
branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+1 💚
mvninstall
0m 33s
the patch passed
+1 💚
compile
0m 34s
the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚
javac
0m 34s
the patch passed
+1 💚
compile
0m 28s
the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+1 💚
javac
0m 28s
the patch passed
+1 💚
blanks
0m 0s
The patch has no blanks issues.
+1 💚
checkstyle
0m 16s
the patch passed
+1 💚
mvnsite
0m 31s
the patch passed
+1 💚
javadoc
0m 31s
the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚
javadoc
0m 46s
the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+1 💚
spotbugs
1m 19s
the patch passed
+1 💚
shadedclient
17m 10s
patch has no errors when building and testing our client artifacts.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of PR
When dfs.federation.router.federation.rename.option is set to DISTCP and hdfs.fedbalance.procedure.scheduler.journal.uri is not set, DFSRouter fails to launch.
hdfs.fedbalance.procedure.scheduler.journal.uri is hdfs://localhost:8020/tmp/procedure by default, however, the default value is not used in DFSRouter.
How was this patch tested?
Local dev testing. After applying fix for HDFS-16219, we can reproduce above error. And the fix provides default value to hdfs.fedbalance.procedure.scheduler.journal.uri.
For code changes:
LICENSE
,LICENSE-binary
,NOTICE-binary
files?