Remove graphs from monitor #5108
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
All of the other pages look the same except the home page which no longer has the graphs:
|
ctubbsii
approved these changes
Nov 25, 2024
3 tasks
DomGarguilo
added a commit
that referenced
this pull request
Nov 27, 2024
PR #5108 removed the graphs from the monitor. A lot of supporting code was able to be removed too. This commit removes one of those methods that was left out of the initial PR.
3 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Partially addresses #5091
This pull request includes several changes focused on removing the Flot library and related code from the project, as well as cleaning up unused imports and methods. The most important changes include the removal of Flot references from various license files, the deletion of Flot-related code from the
Monitor.javaclass, and the removal of corresponding REST endpoints inStatisticsResource.java.Removal of Flot library and related code:
LICENSE,assemble/src/main/resources/LICENSE,server/monitor/src/main/appended-resources/META-INF/LICENSE: Removed references to the Flot library and its dependencies.server/monitor/src/main/resources/org/apache/accumulo/monitor/resources/external/flot/LICENSE.txt: Deleted the Flot license file.Code cleanup in
Monitor.java:newMaxList,fetchData, and various getter methods for metrics over time.Removal of REST endpoints in
StatisticsResource.java:CSS cleanup:
screen.css.