dlmarion commented on code in PR #3482:
URL: https://github.com/apache/accumulo/pull/3482#discussion_r1226799299
##########
server/base/src/main/java/org/apache/accumulo/server/metadata/TabletMutatorBase.java:
##########
@@ -170,15 +170,15 @@ protected String getLocationFamily(LocationType type) {
@Override
public T putLocation(Location location) {
Preconditions.checkState(updatesEnabled, "Cannot make updates after
calling mutate.");
- mutation.put(getLocationFamily(location.getType()), location.getSession(),
- location.getHostPort());
+ mutation.put(getLocationFamily(location.getType()), "",
+ location.getServerInstance().toString());
Review Comment:
That's good information. I don't have the history to know why `sessionId`
was in the column qualifier. It seemed like a bug to me as it would allow
multiple `current`, `future`, or `last` entries.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
