You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "keith-turner (via GitHub)" <gi...@apache.org> on 2023/11/21 16:42:14 UTC

[I] Set default value of table.majc.compaction.strategy property to empty string [accumulo]

keith-turner opened a new issue, #3966:
URL: https://github.com/apache/accumulo/issues/3966

   **Describe the bug**
   
   In 2.1 the default value of the property `table.majc.compaction.strategy` is set to the DefaultCompactionStrategy class. However if the user does not set this property, then its default value is ignored.  If its possible to do w/o breaking anything, the default value should be empty string to better reflect this behavior.
   
   This property was deprecated in 2.x and dropped in 3.0.  so the change only needs to be made in 2.x.


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [I] Set default value of table.majc.compaction.strategy property to empty string [accumulo]

Posted by "ctubbsii (via GitHub)" <gi...@apache.org>.
ctubbsii closed issue #3966: Set default value of table.majc.compaction.strategy property to empty string
URL: https://github.com/apache/accumulo/issues/3966


-- 
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


Re: [I] Set default value of table.majc.compaction.strategy property to empty string [accumulo]

Posted by "keith-turner (via GitHub)" <gi...@apache.org>.
keith-turner commented on issue #3966:
URL: https://github.com/apache/accumulo/issues/3966#issuecomment-1821332810

   > When a user does not set the property, what happens?
   
   It will use the plugin to make compaction decisions.


-- 
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


Re: [I] Set default value of table.majc.compaction.strategy property to empty string [accumulo]

Posted by "EdColeman (via GitHub)" <gi...@apache.org>.
EdColeman commented on issue #3966:
URL: https://github.com/apache/accumulo/issues/3966#issuecomment-1821305009

   When a user does not set the property, what happens?


-- 
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


Re: [I] Set default value of table.majc.compaction.strategy property to empty string [accumulo]

Posted by "keith-turner (via GitHub)" <gi...@apache.org>.
keith-turner commented on issue #3966:
URL: https://github.com/apache/accumulo/issues/3966#issuecomment-1828144628

   >>   When a user does not set the property, what happens?
   >
   > It will use the plugin to make compaction decisions.
   
   I just noticed I misread your question, I missed the `not` in the original question so the answer above is wrong.  When a user does not set this property it will use the newer plugins like its does currently.  Prior to this change Accumulo has the odd behavior of ignoring the default value for this property in the case where the user does not set the property.  So making the default empty string makes it more apparent that the default value is ignored.


-- 
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