datawave
datawave copied to clipboard
Support for further query, query plan formatting (issue #1624)
Changed JexlFormattedStringBuildingVisitor.java to have the option of including HTML for the various query elements. Updated the screen.css file to have support for the HTML elements added in JexlFormattedStringBuildingVisitor (this screen.css file is used in the webserver's Query Metrics page). Moved the call of formatting the metrics to a more appropriate location.
I was hoping that you would use a decorator pattern instead of having if (includeHtml) scattered throughout the code. What that means is that you would have a call to decorator.apply(String text, TYPE type) at the end of each one of those visitor functions. The decorator would implement your own interface that has an apply(String, TYPE) method. Then all of the html stuff would be implemented within an HtmlDecorator. And then the next step would be to create a BashDecorator that would also do the coloring stuff but instead for a bash shell.
Added the script to run JexlFormattedStringBuildingVisitor from the command line.
Script removes line 4 of ingest-libs.sh (". ../ingest/ingest-env.sh") before running it and then adds it back after to avoid error of not finding ingest-passwd.sh. This is probably not the best way to go about this, any ideas of a better way would be great.
Also, the cd command could probably be changed. How can I get the local datawave directory of the user to use that instead?