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 2020/02/11 22:05:52 UTC
[GitHub] [accumulo] milleruntime opened a new issue #1507: Throw Exception
to prevent incorrect metadata
milleruntime opened a new issue #1507: Throw Exception to prevent incorrect metadata
URL: https://github.com/apache/accumulo/issues/1507
Follow on work for #1501. With the new TabletFile, we can now prevent metadata paths being incorrectly written (or not deleted) if an exception is thrown. One way to do this is to have different methods on TabletFile to get the metadata entry depending if it is needed for reading, updating, inserting, or deleting. If the path is needed for reading or inserting, we can use the normalized path. If the path is needed for updating or deleting, then it has to be the original metadata entry (what was read) and TabletFile should throw an error if it is null.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [accumulo] cradal commented on issue #1507: Throw Exception to
prevent incorrect metadata
Posted by GitBox <gi...@apache.org>.
cradal commented on issue #1507: Throw Exception to prevent incorrect metadata
URL: https://github.com/apache/accumulo/issues/1507#issuecomment-591983733
@milleruntime Hello. I was looking at this issue as my possible next task, but I see some things which indicate some work might have already been done. For example, the metadataEntry field in TabletFile errors out if an attempt is made to instantiate it with a null metadataEntry. Hence, it seems that any call to getMetaDataEntry() will return something. Also, getMetadataEntry() is already being called on createDeleteMutation and minor/major compactions. Has this issue already been worked?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [accumulo] milleruntime commented on issue #1507: Throw Exception
to prevent incorrect metadata
Posted by GitBox <gi...@apache.org>.
milleruntime commented on issue #1507: Throw Exception to prevent incorrect metadata
URL: https://github.com/apache/accumulo/issues/1507#issuecomment-591999750
Yes it is almost completed in #1519. Thanks for the ping though I had forgotten I created an 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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [accumulo] milleruntime closed issue #1507: Throw Exception to
prevent incorrect metadata
Posted by GitBox <gi...@apache.org>.
milleruntime closed issue #1507: Throw Exception to prevent incorrect metadata
URL: https://github.com/apache/accumulo/issues/1507
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services