You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2020/09/25 20:31:56 UTC
[GitHub] [incubator-pinot] vincentchenjl edited a comment on pull request #6058: [TE] move dashboard resources for refactoring
vincentchenjl edited a comment on pull request #6058:
URL: https://github.com/apache/incubator-pinot/pull/6058#issuecomment-699129769
> Also, I envision the new coordinator module to be able to access the sql scripts without depending on `thirdeye-dashboard`.
@suvodeep-pyne I agree on this point. However, having all the tests in `thideye-dashboard` and `create-schema.sql` in `thirdeye-pinot` cause `mvn install` to fail. There are two ways going forward I can think of right now. One is to move `create-schema.sql` to `thirdeye-dashboard`, and the other is to move the test cases that require those schema back to `thirdeye-pinot`. Also, the last PR already moved `persistence-local.yml` to `thirdeye-dashboard`, so we also need to move it back if we want to go for second approach. Please let me know if there is any other way to do it.
`[ERROR] beforeMethod(org.apache.pinot.thirdeye.detection.alert.filter.ToAllRecipientsDetectionAlertFilterTest) Time elapsed: 0.003 s <<< FAILURE!
java.lang.RuntimeException: java.io.FileNotFoundException: file:/Users/jianchen/dev-projects/incubator-pinot/thirdeye/thirdeye-pinot/target/thirdeye-pinot-1.0.0-SNAPSHOT.jar!/schema/create-schema.sql (No such file or directory)
at org.apache.pinot.thirdeye.detection.alert.filter.ToAllRecipientsDetectionAlertFilterTest.beforeMethod(ToAllRecipientsDetectionAlertFilterTest.java:77)`
----------------------------------------------------------------
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
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org