New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HDFS-16877: Enables state context for namenode in TestObserverWithRouter #5257
Conversation
🎊 +1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
8c45a52
to
5a68a37
Compare
🎊 +1 overall
This message was automatically generated. |
...bf/src/test/java/org/apache/hadoop/hdfs/server/federation/router/TestObserverWithRouter.java
Outdated
Show resolved
Hide resolved
🎊 +1 overall
This message was automatically generated. |
...-hdfs-rbf/src/main/java/org/apache/hadoop/hdfs/server/federation/router/RouterRpcServer.java
Outdated
Show resolved
Hide resolved
...-hdfs-rbf/src/main/java/org/apache/hadoop/hdfs/server/federation/router/RouterRpcServer.java
Outdated
Show resolved
Hide resolved
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 LGTM
Description of PR
HDFS-16877
We need to set "dfs.namenode.state.context.enabled" to true for the namenode to send it's stateId in client responses.
How was this patch tested?
New test case. Existing test cases still pass.
For code changes: