/**
 * Copyright (C) 2014-2016 LinkedIn Corp. (pinot-core@linkedin.com)
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *         http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */
package com.linkedin.pinot.core.operator;

import com.linkedin.pinot.core.common.Block;
import com.linkedin.pinot.core.common.BlockId;
import com.linkedin.pinot.core.common.Operator;
import com.linkedin.pinot.core.util.trace.TraceContext;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;


/**
 * Any other Pinot Operators should extend BaseOperator
 */
public abstract class BaseOperator implements Operator {
  private static final Logger LOGGER = LoggerFactory.getLogger(BaseOperator.class);

  private final String _operatorName = getOperatorName();

  @Override
  public final Block nextBlock() {
    long start = System.currentTimeMillis();
    Block ret = getNextBlock();
    long end = System.currentTimeMillis();
    LOGGER.trace("Time spent in {}: {}", _operatorName, (end - start));
    TraceContext.logLatency(_operatorName, (end - start));
    return ret;
  }

  @Override
  public final Block nextBlock(BlockId blockId) {
    long start = System.currentTimeMillis();
    Block ret = getNextBlock(blockId);
    long end = System.currentTimeMillis();
    LOGGER.trace("Time spent in {}: {}", _operatorName, (end - start));
    TraceContext.logLatency(_operatorName, (end - start));
    return ret;
  }

  public abstract Block getNextBlock();

  public abstract Block getNextBlock(BlockId blockId);

  // Enforcing sub-class to implement the getOperatorName(), as they can just return a static final,
  // as opposed to this super class calling getClass().getSimpleName().
  public abstract String getOperatorName();

  @Override
  public ExecutionStatistics getExecutionStatistics() {
    throw new UnsupportedOperationException();
  }
}

@@ -1139,67 +1139,8 @@
 );%0A%0A
-  private final String _operatorName = getOperatorName();%0A%0A
   @O
@@ -1139,32 +1139,32 @@
 );%0A%0A  @Override%0A
+
   public final B
@@ -1335,34 +1335,36 @@
 ent in %7B%7D: %7B%7D%22, 
-_o
+getO
 peratorName, (en
@@ -1350,32 +1350,34 @@
  getOperatorName
+()
 , (end - start))
@@ -1398,34 +1398,36 @@
 text.logLatency(
-_o
+getO
 peratorName, (en
@@ -1413,32 +1413,34 @@
 (getOperatorName
+()
 , (end - start))
@@ -1692,18 +1692,20 @@
 %7D: %7B%7D%22, 
-_o
+getO
 peratorN
@@ -1699,32 +1699,34 @@
  getOperatorName
+()
 , (end - start))
@@ -1759,10 +1759,12 @@
 ncy(
-_o
+getO
 pera
@@ -1770,16 +1770,18 @@
 atorName
+()
 , (end -

Remove redundant data member from BaseOperator class. (#1003)