You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2021/07/28 12:35:02 UTC
[GitHub] [accumulo-testing] milleruntime opened a new issue #145: Update CI to use Compaction Planner
milleruntime opened a new issue #145:
URL: https://github.com/apache/accumulo-testing/issues/145
Continuous Ingest is still using `BasicCompactionStrategy`, which has been deprecated in favor of the new compaction code. This needs to be replaced with a CompactionPlanner, most likely `DefaultCompactionPlanner` will work fine. This may just be a configuration change.
This would be great to do for the 2.1 release testing.
--
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: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo-testing] DomGarguilo commented on issue #145: Update CI to use Compaction Planner
Posted by GitBox <gi...@apache.org>.
DomGarguilo commented on issue #145:
URL: https://github.com/apache/accumulo-testing/issues/145#issuecomment-898664971
It does seem like the tables default to use `DefaultCompactionPlanner`. Do you think its a good idea to still try and set them explicitly in the props file or just remove the deprecated `BasicCompactionStrategy`? @milleruntime
--
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: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo-testing] milleruntime commented on issue #145: Update CI to use Compaction Planner
Posted by GitBox <gi...@apache.org>.
milleruntime commented on issue #145:
URL: https://github.com/apache/accumulo-testing/issues/145#issuecomment-888271015
It may be as simple as changing these properties:
https://github.com/apache/accumulo-testing/blob/main/conf/accumulo-testing.properties#L44-L47
--
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: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo-testing] DomGarguilo closed issue #145: Update CI to use Compaction Planner
Posted by GitBox <gi...@apache.org>.
DomGarguilo closed issue #145:
URL: https://github.com/apache/accumulo-testing/issues/145
--
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: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org