You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@phoenix.apache.org by GitBox <gi...@apache.org> on 2020/09/17 20:23:32 UTC
[GitHub] [phoenix] tamaashu opened a new pull request #887: PHOENIX-6143: Get Phoenix Tracing Webapp work
tamaashu opened a new pull request #887:
URL: https://github.com/apache/phoenix/pull/887
Tracing webapp was broken for a while, these changes get it work.
Change-Id: I81b20d9abff220b01a984af3722ee57fef316621
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [phoenix] stoty commented on a change in pull request #887: PHOENIX-6143: Get Phoenix Tracing Webapp work
Posted by GitBox <gi...@apache.org>.
stoty commented on a change in pull request #887:
URL: https://github.com/apache/phoenix/pull/887#discussion_r493232427
##########
File path: phoenix-tracing-webapp/pom.xml
##########
@@ -155,6 +131,7 @@
<plugin>
<groupId>com.github.searls</groupId>
<artifactId>jasmine-maven-plugin</artifactId>
+ <version>2.2</version>
Review comment:
You are overriding the version set in the master pom pluginManagement section.
I suggest that you override the plugin version there instead.
##########
File path: phoenix-tracing-webapp/src/build/trace-server-runnable.xml
##########
@@ -43,7 +43,19 @@
<outputDirectory>/</outputDirectory>
<includes>
<include>org.apache.phoenix:phoenix-tracing-webapp</include>
Review comment:
Ideally, this should be done by the shade plugin instead. (Not required for this patch, just a remark)
##########
File path: phoenix-tracing-webapp/README.md
##########
@@ -3,10 +3,10 @@
*mvn clean install*
Review comment:
Nit: We use backticks to format command lines elsewhere, might as well switch to that here, as well.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [phoenix] stoty closed pull request #887: PHOENIX-6143: Get Phoenix Tracing Webapp work
Posted by GitBox <gi...@apache.org>.
stoty closed pull request #887:
URL: https://github.com/apache/phoenix/pull/887
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org