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 2022/09/01 17:55:46 UTC
[GitHub] [accumulo] ctubbsii opened a new pull request, #2911: Serialize location in BadLocationStateException
ctubbsii opened a new pull request, #2911:
URL: https://github.com/apache/accumulo/pull/2911
To ensure that the location is correctly serialized in this exception,
do not mark it transient, but convert it to a byte array, since Text is
not Serializable.
This addresses the warning in #2776 without assuming the exception will
never be serialized, which was made in #2897. Instead, this change
avoids errors if it ever were to be serialized and the location were
silently changed to a null value because the field was transient.
--
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] ctubbsii merged pull request #2911: Serialize location in BadLocationStateException
Posted by GitBox <gi...@apache.org>.
ctubbsii merged PR #2911:
URL: https://github.com/apache/accumulo/pull/2911
--
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