ayushtkn commented on PR #7112: URL: https://github.com/apache/hadoop/pull/7112#issuecomment-2448228790
Fundamentally to me that seems wrong only, any implementation is free to have different logic for `null` & different for empty map & we don't know at the Hadoop layer what all implements that or comment about internal implementations... Moreover for the ones doing the ``null`` check already for them it would be an extra redundant check. > It is complex process to fix it in AmqPlainSaslClientFactory, release qpid and then change the qpid version within our application you have to patch hadoop, release it & change that version in the client, that goes with any code change in any layer, this can't be a reason to do a change in Hadoop :-) -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
