DomGarguilo commented on PR #42: URL: https://github.com/apache/accumulo-proxy/pull/42#issuecomment-1317624224
There are a few side affects from the changes in this PR. Some good, some bad. The two ITs that were failing previously now pass (#37 & #39). But now two other ITs are failing `testSiteConfiguration` and `userAuthentication`. Also I think logging is broken with this change due to log4j/slf4j conflicts. I can try to fix and incorporate the things that are broken into this PR or, since its already large, and to limit the scope of this PR, merge this as is and create follow on tickets for them. -- 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]
