Jens-G commented on PR #3606: URL: https://github.com/apache/thrift/pull/3606#issuecomment-4845895077
One housekeeping note: for a change of this scope, it is worth making sure the JIRA ticket captures the key design decisions and rationale in addition to what is in the commit messages. Commit messages are squashed on merge and can be hard to find later; JIRA comments and the ticket description remain a stable reference for anyone reviewing the issue in future. If there is technical context currently spread across individual commit messages (e.g. the rationale for the `takeOwnership` parameter semantics, or the TLS version floor decision), copying the relevant parts into the JIRA ticket before this is merged would be a good investment. -- 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]
