akshayrai commented on a change in pull request #3830: [TE] Presto Connector
backend and front end
URL: https://github.com/apache/incubator-pinot/pull/3830#discussion_r256931454
##########
File path:
thirdeye/thirdeye-frontend/app/pods/self-serve/create-alert/template.hbs
##########
@@ -29,6 +29,9 @@
<label for="select-metric" class="control-label te-label
te-label--taller required">Search for a metric, or
{{#link-to "self-serve.import-metric" class="thirdeye-link-secondary
thirdeye-link-secondary--inside"}}
Import a Metric From InGraphs
+ {{/link-to}} or
+ {{#link-to "self-serve.import-presto-metric"
class="thirdeye-link-secondary thirdeye-link-secondary--inside"}}
+ Import a Metric From Presto
Review comment:
I am not sure if we should surface this yet! This will make the link visible
in production. Maybe comment this code out for now and we can enable it for the
ThirdEye users once we test it internally and are happy with the latency
requirements.
Also make sure the public facing links are suppressed elsewhere in this PR
if any.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
