You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "dlmarion (via GitHub)" <gi...@apache.org> on 2023/04/17 21:43:51 UTC

[GitHub] [accumulo] dlmarion opened a new issue, #3310: OnDemand Follow-on: Resolve how Tablet.java will stay in sync with metadata changes

dlmarion opened a new issue, #3310:
URL: https://github.com/apache/accumulo/issues/3310

   Specifically, resolve todo at line 236 in Tablet.java. The hostingGoal could be updated by a user (ALWAYS <-> ONDEMAND) while the Tablet is hosted and in-memory. I don't think we need to worry about the NEVER case, as that would cause the Tablet to become unhosted.
   
   Generally, tablet management functions that are moved to the Manager (e.g. Bulk Import, Compaction, etc) will likely modify the TabletMetadata. For any hosted Tablet, how will the Tablet object stay in sync?


-- 
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] OnDemand Follow-on: Resolve how Tablet.java will stay in sync with metadata changes [accumulo]

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

   > #3350 added TabletServerClientService.refreshTablets. I assume we just need to add a call to this in the correct places to resolve this issue?
   
   When refresh was first added it had race conditions on the tserver side. Those were cleaned up and fixed in #3625 and #3634.  There is still work that could be done to clean up the overall locking in the tserver, which is outlined in #3670.  Cleaning up locking the in the tablet would be good since compactions and bulk import were removed and the tablet code is simpler.


-- 
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] OnDemand Follow-on: Resolve how Tablet.java will stay in sync with metadata changes [accumulo]

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

   #3350 added `TabletServerClientService.refreshTablets`. I assume we just need to add a call to this in the correct places to resolve this issue?


-- 
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] OnDemand Follow-on: Resolve how Tablet.java will stay in sync with metadata changes [accumulo]

Posted by "keith-turner (via GitHub)" <gi...@apache.org>.
keith-turner closed issue #3310: OnDemand Follow-on: Resolve how Tablet.java will stay in sync with metadata changes
URL: https://github.com/apache/accumulo/issues/3310


-- 
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 #3310: OnDemand Follow-on: Resolve how Tablet.java will stay in sync with metadata changes

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

   May be easier to work on this after functionality is removed from the tablet server.  Then we will only need to update the code that is left behind in the tablet server.


-- 
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] OnDemand Follow-on: Resolve how Tablet.java will stay in sync with metadata changes [accumulo]

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

   Also compaction commit has a customized refresh processes that could be simplified by using FATE as outlined in #3559


-- 
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] OnDemand Follow-on: Resolve how Tablet.java will stay in sync with metadata changes [accumulo]

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

   Closing this issue, the work is done and follow on issue are open for potential improvements.


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