You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "cshannon (via GitHub)" <gi...@apache.org> on 2024/02/09 20:39:54 UTC
[I] Rename FATE AccumuloStore to something else more descriptive [accumulo]
cshannon opened a new issue, #4253:
URL: https://github.com/apache/accumulo/issues/4253
#4049 introduce a new FATE store type to store fate transactions in an Accumulo table. Right now it is called AccumuloStore but we should rename it to be more descriptive and less confusing as to what it is. Ideally it would be good to leave "Accumulo" out of the name since the entire project is called Accumulo. One possible option could be `TableFateStore` and then rename ZooStore to `ZookeperFateStore` (or `ZooFateStore`).
--
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] Rename FATE AccumuloStore to something else more descriptive [accumulo]
Posted by "dlmarion (via GitHub)" <gi...@apache.org>.
dlmarion commented on issue #4253:
URL: https://github.com/apache/accumulo/issues/4253#issuecomment-1938904490
Is the Zoo store is for system initiated FATE transactions, and the Table store is for user initiated FATE transactions, or is it the case that the Zoo store is for FATE transactions on the System tables, and the Table store is for FATE transactions on user tables?
--
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] Rename FATE AccumuloStore to something else more descriptive [accumulo]
Posted by "cshannon (via GitHub)" <gi...@apache.org>.
cshannon commented on issue #4253:
URL: https://github.com/apache/accumulo/issues/4253#issuecomment-1939227846
> So, maybe instead of `ZooStore` and `AccumuloStore`, name them `SystemTableFateStore` and `UserTableFateStore`.
Sounds fine to me unless anyone else has any suggestions. I think the plan is to wait until all the outstanding FATE stuff to be merged and then we can do the rename to not break any open PRs
--
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] Rename FATE AccumuloStore to something else more descriptive [accumulo]
Posted by "cshannon (via GitHub)" <gi...@apache.org>.
cshannon closed issue #4253: Rename FATE AccumuloStore to something else more descriptive
URL: https://github.com/apache/accumulo/issues/4253
--
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] Rename FATE AccumuloStore to something else more descriptive [accumulo]
Posted by "cshannon (via GitHub)" <gi...@apache.org>.
cshannon commented on issue #4253:
URL: https://github.com/apache/accumulo/issues/4253#issuecomment-1939072977
It's the latter: The Zoo store is only going to be used to store FATE transactions for system tables (root, metadata, etc). Everything else (for now at least, more below) goes into the Table store for other tables so that performance is better now that Accumulo will generate a lot more FATE operations for all the system initiated transactions.
There was discussion with @keith-turner about splitting up the Table store into 2 stores (or 2 sections of the table) to separate out user vs system initiated operations but for now we decided to just put everything into one Table store and see how it goes. So we could end up with a 3rd store at some point if we need it (system tables in Zoo store, Table store for Users ops and Table store for system ops for all other tables)
--
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] Rename FATE AccumuloStore to something else more descriptive [accumulo]
Posted by "cshannon (via GitHub)" <gi...@apache.org>.
cshannon commented on issue #4253:
URL: https://github.com/apache/accumulo/issues/4253#issuecomment-2002087181
I talked to @keith-turner and I am going to go with `MetaFateStore` and `UserFateStore` since it aligns with the `FateInstanceType` enum.
--
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] Rename FATE AccumuloStore to something else more descriptive [accumulo]
Posted by "dlmarion (via GitHub)" <gi...@apache.org>.
dlmarion commented on issue #4253:
URL: https://github.com/apache/accumulo/issues/4253#issuecomment-1939078956
So, maybe instead of `ZooStore` and `AccumuloStore`, name them `SystemTableFateStore` and `UserTableFateStore`.
--
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