[GitHub] [skywalking] wu-sheng commented on pull request #10448: Integrate BanyanDB server-side TopN

["wu-sheng (via GitHub)" <gi...@apache.org>]

wu-sheng commented on PR #10448:
URL: https://github.com/apache/skywalking/pull/10448#issuecomment-1445380385

   > Now we have to decide how we can define the parameters? May be in the annotation?
   
   I think BanyanDB's specific annotation works. 
   
   > And another issue is to support additional filter in the aggregation query. The only property is serviceID now?
   
   For now, I think only the time range, max candidates for precalculation and service ID.


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@skywalking.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org