smolnar82 commented on PR #1456: URL: https://github.com/apache/knox/pull/1456#issuecomment-6231914935
> Thanks Sandor. It looks good to me. In a normal production state, refreshing the cache after ttl/2 makes sense. I'm concerned if there are problems with the LB or in a test environment if one node might be idle for a while and then test startup might see failures for the first few queries until it starts to work. Possibly this is fine, just warm before test, and simpler than an active poll. I'm not sure how much harder an active poll would be. I think we can investigate that in the scope of a follow-up JIRA. I'll read your comments and submit a new patchset soon. -- 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]
