siddharthteotia commented on a change in pull request #5774:
URL: https://github.com/apache/incubator-pinot/pull/5774#discussion_r465941967



##########
File path: 
pinot-controller/src/main/java/org/apache/pinot/controller/recommender/rules/AbstractRule.java
##########
@@ -16,22 +16,19 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.pinot.tools.tuner.query.src.parser;
+package org.apache.pinot.controller.recommender.rules;
 
-import javax.annotation.Nullable;
-import org.apache.pinot.tools.tuner.query.src.stats.wrapper.AbstractQueryStats;
+import org.apache.pinot.controller.recommender.io.ConfigManager;
+import org.apache.pinot.controller.recommender.io.InputManager;
 
 
-/**
- * Parser interface for a query line
- */
-public interface QueryParser {
-  /**
-   * parse the the complete log line to a parsed obj
-   * @param line the complete log line to be parsed, InputIterator should put 
broken lines together
-   * @return the parsed log line obj
-   */
-  @Nullable
-  AbstractQueryStats parse(String line);
-}
+public abstract class AbstractRule {

Review comment:
       Got it. They came from tuner




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

Reply via email to