NihalJain commented on PR #6585: URL: https://github.com/apache/hbase/pull/6585#issuecomment-2594661076
> IMO HBase already has more than enough dependencies, we should try to avoid adding more if we can. > > Is there a known security issue with the current xmlEscape ? Is there something in another existing dependency that we could use ? +1 on idea of avoiding new dependency. I too am skeptical about this new addition and needed another set of eyes to approve of this. Thanks for chiming in @stoty ! -- 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]
