pvary commented on PR #3371: URL: https://github.com/apache/hive/pull/3371#issuecomment-1167123846
> > > If we changed 45 to ... like 450, the substring can only show zhe zookeeper connection url we input like host1:2181,host2:2181,host3:2181. We still can't get server host. For example hiveserver2host:10000 > > > > > > The question is what do we want to see here: > > > > * The URI which could be used to reconnect to the cluster to rerun the queries > > * The specific server's URI to which we are connected to > > As I see, I don't think it can be used to reconnect to the cluster. > > * The substring is not a complete connection URI, it only has host with port. > * For some long URIs like zookeeper connection with many hosts, 45 char long probably can't show the whole host information. So this is a marker only for the user to help identify the session they are using. I think having part of the original string there is more helpful than having a "somewhat unrelated" string there, even when I see why it is so confusing in your case. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
