dlmarion commented on code in PR #5994:
URL: https://github.com/apache/accumulo/pull/5994#discussion_r2578527460
##########
core/src/main/java/org/apache/accumulo/core/client/admin/compaction/ErasureCodeConfigurer.java:
##########
@@ -85,9 +85,12 @@ public Overrides override(InputParameters params) {
long inputsSum =
params.getInputFiles().stream().mapToLong(CompactableFile::getEstimatedSize).sum();
if (inputsSum >= this.ecSize) {
- overs.put(Property.TABLE_ENABLE_ERASURE_CODES.getKey(), "enable");
- if (ecPolicyName != null) {
- overs.put(Property.TABLE_ERASURE_CODE_POLICY.getKey(), ecPolicyName);
+ if (!params.getEnvironment().getConfiguration(params.getTableId())
Review Comment:
@kevinrr888 - I didn't notice that you were assigned #5960 , so I put up a
quick fix in #5995. I'm fine with closing my PR, but I think we might need to
check the table configuration for the presence of the
Property.TABLE_ERASURE_CODE_POLICY. If it's not set in the table configuration
and not set in the configuration for this class, then the error will still be
raised I think.
--
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]
