[
https://issues.apache.org/jira/browse/HDFS-14447?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16840643#comment-16840643
]
Íñigo Goiri commented on HDFS-14447:
------------------------------------
Can you fix the whitespace?
There is a couple things broken:
{code}
assertNotEquals("Should be different group: " + g1.get(i) + " and " +
g3.get(i), g1.get(i).equals(g3.get(i)));
{code}
It's doing a comparisson with the string, this should be:
{code}
assertNotEquals("Should be different group: " + g1.get(i) + " and " +
g3.get(i), g1.get(i), g3.get(i));
{code}
Then, the log line like:
{code}
LOG.info(g4.toString());
{code}
Should be:
{code}
LOG.info("Group 4: {}", g4);
{code}
In the writer in 362, it would be better to do
{{writer.println(newResource.toString())}} to make sure the StrinbBuilder uses
the {{toString()}}.
The javadoc in {{testRefreshSuperUserGroupsConfigurationInternal()}} should not
finish with a double asterisk.
> RBF: Router should support RefreshUserMappingsProtocol
> ------------------------------------------------------
>
> Key: HDFS-14447
> URL: https://issues.apache.org/jira/browse/HDFS-14447
> Project: Hadoop HDFS
> Issue Type: Sub-task
> Components: rbf
> Affects Versions: 3.1.0
> Reporter: Shen Yinjie
> Assignee: Shen Yinjie
> Priority: Major
> Fix For: HDFS-13891
>
> Attachments: HDFS-14447-HDFS-13891.01.patch,
> HDFS-14447-HDFS-13891.02.patch, HDFS-14447-HDFS-13891.03.patch,
> HDFS-14447-HDFS-13891.04.patch, HDFS-14447-HDFS-13891.05.patch,
> HDFS-14447-HDFS-13891.06.patch, HDFS-14447-HDFS-13891.07.patch, error.png
>
>
> HDFS with RBF
> We configure hadoop.proxyuser.xx.yy ,then execute hdfs dfsadmin
> -Dfs.defaultFS=hdfs://router-fed -refreshSuperUserGroupsConfiguration,
> it throws "Unknown protocol: ...RefreshUserMappingProtocol".
> RouterAdminServer should support RefreshUserMappingsProtocol , or a proxyuser
> client would be refused to impersonate.As shown in the screenshot
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
