ddanielr commented on PR #5807:
URL: https://github.com/apache/accumulo/pull/5807#issuecomment-3197453753

   > For reference, the root and metadata table configuration are set to use 
`root` and `meta` in the 2.1 InitialConfiguration class 
[here](https://github.com/apache/accumulo/blob/2.1/server/base/src/main/java/org/apache/accumulo/server/init/InitialConfiguration.java#L63-L65)
 and 
[here](https://github.com/apache/accumulo/blob/2.1/server/base/src/main/java/org/apache/accumulo/server/init/InitialConfiguration.java#L102-L104).
 In 4.0 those configuration properties are not set on the root and metadata 
tables in InitialConfiguration.
   
   In 4.0, the 
[CompactionJobPrioritizer](https://github.com/apache/accumulo/blob/582da789ac97a56fa313fa71fe109eb8f44bafc9/core/src/main/java/org/apache/accumulo/core/util/compaction/CompactionJobPrioritizer.java#L37-L36)
 was updated to determine the correct compaction job priority across all 
tables, removed the need to have separate meta and root compaction services. 
   
   I still think the current approach of failing the upgrade is the best 
solution to avoid unexpected behavior.
   
   


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

Reply via email to