Github user ajs6f commented on the issue:
    Holy moly, @afs , thanks for the huge assist! I was out yesterday (Yom 
Kippur) and am now catching up. I will leave comments specific to the HTTP 
question on the new PR, but I will try to speak to the other points here.
    1. I am certainly happy to see a spurious plugin go. I have no problem with 
including that change in this package of changes, if that's okay with everyone 
else. Or it could be a quick separate ticket/commit, to keep thing straighter 
in the history.
    2. I have no idea why `maven-source-plugin ` is overridden. Baggage from 
long-ago when this material was imported into the main code base?
    3. I haven't seen AspectJ used anywhere, but I haven't been all through the 
`jdbc` code. Same as the first point-- I have no problem removing it here or 

If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at or file a JIRA ticket
with INFRA.

Reply via email to