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/12/01 19:09:04 UTC

[GitHub] [accumulo] DomGarguilo opened a new issue #2373: Push tablet partitioning based on DataLevel to Ample

DomGarguilo opened a new issue #2373:
URL: https://github.com/apache/accumulo/issues/2373


   In  #2320 we partition tablets based on their DataLevel into two groups: one set who's DataLevel is USER and another containing the remaining tablets (those who's DataLevel is ROOT or METADATA). This needs to happen so we can use `.readTablets()` on the DataLevel.USER tablets in order to reduce RPCs.
   
   This partitioning logic should be pushed to ample.


-- 
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] DomGarguilo commented on issue #2373: Push tablet partitioning based on DataLevel to Ample

Posted by GitBox <gi...@apache.org>.
DomGarguilo commented on issue #2373:
URL: https://github.com/apache/accumulo/issues/2373#issuecomment-996887428


   > @DomGarguilo are you working on this still? I was thinking of taking a look into it.
   
   Go ahead, I took a quick look but then got sidetracked.


-- 
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] keith-turner closed issue #2373: Push tablet partitioning based on DataLevel to Ample

Posted by GitBox <gi...@apache.org>.
keith-turner closed issue #2373:
URL: https://github.com/apache/accumulo/issues/2373


   


-- 
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] DomGarguilo commented on issue #2373: Push tablet partitioning based on DataLevel to Ample

Posted by GitBox <gi...@apache.org>.
DomGarguilo commented on issue #2373:
URL: https://github.com/apache/accumulo/issues/2373#issuecomment-987022336


   For context, here is the comment where this change was first suggested:
   
   https://github.com/apache/accumulo/blob/6cfb9180a0d3e5115922314ff2062e0706ef0795/core/src/main/java/org/apache/accumulo/core/metadata/schema/TabletsMetadata.java#L324-L328
   
   So if the tablet server was serving the root or metadata tablets, then it would throw an exception for this.  Could partition the tablets into user and non-user (root,metadata) tablets.  Can use readTablets() w/ the user tablets.  For the non-user would have to call readTablet() one at a time.  Could do that partitioning in the tablet server code for now and in a later PR push it down into Ample and remove it from the tablet server code.
   
   _Originally posted by @keith-turner in https://github.com/apache/accumulo/pull/2320#discussion_r747057650_


-- 
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] keith-turner commented on issue #2373: Push tablet partitioning based on DataLevel to Ample

Posted by GitBox <gi...@apache.org>.
keith-turner commented on issue #2373:
URL: https://github.com/apache/accumulo/issues/2373#issuecomment-996886346


   @DomGarguilo are you working on this still? I was thinking of taking a look into it.


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