Hi Andy, Andriy
Re CXF-7070, I'd rather see the relevant changes linked to CXF-7070, it
is about not logging the headers which are considered to be sensitive
for a given production, which is not related to the performance related
enhancements and it will be easier to discuss these orthogonal changes.
I'm not sure right now how to separate the commits from a single PR, but
I guess it is not really a problem at all, as far as CXF-7075 is
concerned we know we are now looking only at possibly optimizing some of
JAXRSUtils code - I'll simply close this PR once it is finalized.
Andy, can you please open a new PR against CXF-7070 ?
On 11/10/16 02:18, Andriy Redko wrote:
I think in this case, that would be more of a preference question, your commit
is pretty small one (just 5 files). You may keep 3 commits or just squash them
into 1 commit,
AM> Sergey, All,
AM> While waiting on the performance benchmark results for 7075, I thought I
AM> would take a look at 7070. I ended up committing changes to that under the
AM> same pull request that is still open for 7075 - different commit, but same
AM> I'm still coming up to speed on Git, so there is probably an easy way to
AM> separate the different commits into different PRs, but I don't know how
AM> offhand. Do you know how to do this? Since they are different commits
AM> does it matter if it is in the same PR or not? Sorry for the rookie