priyankporwal commented on a change in pull request #801:
URL: https://github.com/apache/phoenix/pull/801#discussion_r440287703
##########
File path:
phoenix-core/src/main/java/org/apache/phoenix/coprocessor/IndexRebuildRegionScanner.java
##########
@@ -91,13 +95,15 @@
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-import com.google.common.base.Throwables;
import com.google.common.collect.Maps;
public class IndexRebuildRegionScanner extends GlobalIndexRegionScanner {
private static final Logger LOGGER =
LoggerFactory.getLogger(IndexRebuildRegionScanner.class);
+ public static final String
PHOENIX_LOG_BEYOND_MAX_LOOKBACK_ERROR_ROWS_CONF_KEY =
+ "phoenix.log.beyond.max.lookback.age.error.rows";
Review comment:
Can we please have this config be named consistently like the previous
one?
"phoenix.mr.index.IndexDisableLoggingType=BEFORE" .. It would make it easier
to find and put those together whenever they need non-default values.
----------------------------------------------------------------
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]
