Skip to content
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

Consolidate webserver to just NameNodeAnalyticsHttpServer #222

Open
pjeli opened this issue Apr 15, 2019 · 1 comment
Open

Consolidate webserver to just NameNodeAnalyticsHttpServer #222

pjeli opened this issue Apr 15, 2019 · 1 comment
Labels
documentation Documentation needed help wanted Extra attention is needed reduction Reduces or simplifies code
Milestone

Comments

@pjeli
Copy link
Collaborator

pjeli commented Apr 15, 2019

Keeping in line with getting to Apache adoption, the next major refactoring stretch is to remove the old SparkJava webserver and replace it with the Jetty-based NameNodeAnalyticsHttpServer (which is just an extension / minor re-write of the Apache Hadoop NameNode webserver wrapper, NameNodeHttpServer).

This will lessen our codebase and make it even easier to manage.

@pjeli pjeli added this to the 1.6.3 Release milestone Apr 15, 2019
@pjeli
Copy link
Collaborator Author

pjeli commented Apr 15, 2019

The trickiest part of this is validating that SSL and all APIs work exactly the same as the old SparkJava based webserver. So far I have been covering that with unit tests but until I do a swap with an existing instance I cannot validate it safely.

@pjeli pjeli added documentation Documentation needed help wanted Extra attention is needed reduction Reduces or simplifies code labels Apr 15, 2019
@pjeli pjeli modified the milestones: 1.6.3 Release, ASF Induction Apr 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Documentation needed help wanted Extra attention is needed reduction Reduces or simplifies code
Projects
None yet
Development

No branches or pull requests

1 participant