dlmarion commented on code in PR #5924:
URL: https://github.com/apache/accumulo/pull/5924#discussion_r2379594071
##########
core/src/main/java/org/apache/accumulo/core/constraints/DefaultKeySizeConstraint.java:
##########
@@ -38,43 +36,18 @@
public class DefaultKeySizeConstraint extends
org.apache.accumulo.core.data.constraints.DefaultKeySizeConstraint
implements Constraint {
- protected static final short MAX__KEY_SIZE_EXCEEDED_VIOLATION = 1;
- protected static final long maxSize = 1048576; // 1MB default size
+ protected static final short MAX__KEY_SIZE_EXCEEDED_VIOLATION =
+
org.apache.accumulo.core.data.constraints.DefaultKeySizeConstraint.MAX__KEY_SIZE_EXCEEDED_VIOLATION;
+ protected static final long maxSize =
+
org.apache.accumulo.core.data.constraints.DefaultKeySizeConstraint.maxSize;
@Override
public String getViolationDescription(short violationCode) {
-
- switch (violationCode) {
- case MAX__KEY_SIZE_EXCEEDED_VIOLATION:
- return "Key was larger than 1MB";
- }
-
- return null;
+ return super.getViolationDescription(violationCode);
Review Comment:
Removed this method in 6fd5db0
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
