Skip to content
This repository has been archived by the owner on Nov 16, 2023. It is now read-only.

Pylon: replace WehHDFS URLs in REST server responses #1888

Merged
merged 1 commit into from Dec 17, 2018
Merged

Pylon: replace WehHDFS URLs in REST server responses #1888

merged 1 commit into from Dec 17, 2018

Conversation

Gerhut
Copy link
Member

@Gerhut Gerhut commented Dec 14, 2018

Fixes #1887

@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 51.774% when pulling ae7331a on qixcheng/pylon/replace-webhdfs-in-rest-server into 54cbf1c on master.

fanyangCS
fanyangCS approved these changes Dec 15, 2018
View reviewed changes
Copy link
Contributor

@fanyangCS fanyangCS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we consider 50070 that legacy request to pylon?

@Gerhut
Copy link
Member Author

Gerhut commented Dec 16, 2018 •
edited

@fanyangCS

50070 is a pure transparent proxy to 5070 without any response replacements, like request the WebHDFS api directly but not through pylon.

@Gerhut Gerhut merged commit 1159766 into master Dec 17, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@fanyangCS fanyangCS fanyangCS approved these changes

@wangcan0329 wangcan0329 Awaiting requested review from wangcan0329

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@Gerhut @coveralls @fanyangCS