[GitHub] [bookkeeper] dlg99 commented on pull request #1672: Issue #850: Added request context across client and bookies.

[GitHub]

I'll leave this the way it is.
createExecutor() is called in the constructor so changing it to the local value 
creates expectations for the order of the value initialization.  
Performance-wise it is a noop since it is only used in the constructor.

[ Full content available at: https://github.com/apache/bookkeeper/pull/1672 ]
This message was relayed via gitbox.apache.org for devnull@infra.apache.org