[HDFS-12579] JournalNodeSyncer should use fromUrl field of EditLogManifestResponse to construct servlet Url

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.1.0
• Hadoop Flags: Reviewed
• Assignee: Hanisha Koneru
• Reporter: Hanisha Koneru
• Created: 02/Oct/17 22:58
• Updated: 26/Oct/17 01:22
• Resolved: 25/Oct/17 21:25

Description

Currently in JournalNodeSyncer, we construct the remote JN http server url using the JN host address and the http port that we get from the GetEditLogManifestResponseProto.

          if (remoteJNproxy.httpServerUrl == null) {
            remoteJNproxy.httpServerUrl = getHttpServerURI("http",
                remoteJNproxy.jnAddr.getHostName(), response.getHttpPort());
          }

The correct way would be to get the http server url of the remote JN from the fromUrl field of the GetEditLogManifestResponseProto. This would take care of the http policy set on the remote JN as well.