[GitHub] [incubator-pinot] mcvsubbu commented on pull request #6255: Creating a pluggable interface for Table config tuner

[GitBox <gi...@apache.org>]

mcvsubbu commented on pull request #6255:
URL: https://github.com/apache/incubator-pinot/pull/6255#issuecomment-733862418


   As pointed out by @siddharthteotia  we already have a rule engine in place.
   We also have a realtime prov helper that accepts a sample segment.
   
   Can we follow the interface/factory pattern here, so that we can ingest whatever factory we want? That way, if a rule engine needs more information (sample segments, sample queries, etc) to make the decision, it can.
   
   Also, we should be able to chain these (and each member of the chain should modify only those that are not already touched). 


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