abstractdog commented on code in PR #308:
URL: https://github.com/apache/tez/pull/308#discussion_r1361625385
##########
tez-api/src/main/java/org/apache/tez/dag/api/TezConfiguration.java:
##########
@@ -1580,6 +1580,26 @@ public TezConfiguration(boolean loadDefaults) {
TEZ_PREFIX + "generate.debug.artifacts";
public static final boolean TEZ_GENERATE_DEBUG_ARTIFACTS_DEFAULT = false;
+ /**
+ * Int value. Property size threshold (in bytes) for logging during payload
serialization. Properties exceeding the
+ * threshold are considered unusually large and potentially problematic thus
they should be logged.
+ */
+ @ConfigurationScope(Scope.VERTEX)
+ @ConfigurationProperty(type="integer")
+ public static final String TEZ_LOGGING_PROPERTY_SIZE_THRESHOLD =
+ TEZ_PREFIX + "logging.property.size.threshold";
+ public static final int TEZ_LOGGING_PROPERTY_SIZE_THRESHOLD_DEFAULT = 512 *
1024;
+ /**
Review Comment:
nit, we tend to add a line between every config properties, even if they are
related to each other
--
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: issues-unsubscr...@tez.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
