milleruntime commented on issue #1317: URL: https://github.com/apache/accumulo/issues/1317#issuecomment-724172837
> For my previous comment, the Location type will not really work for this case. Need a type that just encapsulates `(server,session)` for the set of live tservers. TabletMetadata.Location has those plus LocationType, which makes no sense for a live tserver. What do you think about a new type in core.metadata (similar to what I did for TabletFile) that could be used everywhere? And would we want to keep the ZK session Id? @EdColeman and I were talking about the usefulness of storing the internal session ID. It seems like its used as a unique identifier of the ephemeral ZooLock for that tserver. This is a separate issue but I don't think we would want to have the session in this new type. ---------------------------------------------------------------- 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: [email protected]
