vincentchenjl commented on a change in pull request #5823:
URL: https://github.com/apache/incubator-pinot/pull/5823#discussion_r467220926
##########
File path:
thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/dashboard/resources/AnomalyResource.java
##########
@@ -156,30 +156,49 @@ public MergedAnomalyResultDTO getMergedAnomalyDetail(
* eg. payload
* <p/>
* { "feedbackType": "NOT_ANOMALY", "comment": "this
is not an anomaly" }
+ * @param propagate : a flag whether it should propagate the same
feedback value to the parent of this anomaly
+ * and all its siblings if they exist
*/
@POST
@Path(value = "anomaly-merged-result/feedback/{anomaly_merged_result_id}")
@ApiOperation("update anomaly merged result feedback")
- public void
updateAnomalyMergedResultFeedback(@PathParam("anomaly_merged_result_id") long
anomalyResultId,
+ public void updateAnomalyMergedResultFeedback(
+ @PathParam("anomaly_merged_result_id") long anomalyResultId,
+ @QueryParam("propagate") @DefaultValue("true") boolean propagate,
Review comment:
I confirm that updating feedbacks in anomaly pages, root cause pages,
application pages and alert pages will trigger this API call. I don't think
there is any other page where we can update the feedback.
----------------------------------------------------------------
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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
