You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I think broadway_sqs should not require saxy at all. It should at least be marked as optional (or perhaps limited to :dev and/or :test). It doesn't seem that this project has any explicit dependency on saxy AFAICT.
The text was updated successfully, but these errors were encountered:
I may be misrembering but I was the one who added Saxy support to ex_aws_sqs because performance is much better. If saxy is not listed, it defaults to something xmerl or something else, which ends up being slower. So us defaulting to that is a way for us to guarantee good performance out of the box.
Saxy seems to be a transient dependency for ExAwsSQS
Saxy itself is marked as optional in ExAwsSQS V3.2.1+
https://github.com/ex-aws/ex_aws_sqs/blob/main/CHANGELOG.md
I think broadway_sqs should not require saxy at all. It should at least be marked as optional (or perhaps limited to :dev and/or :test). It doesn't seem that this project has any explicit dependency on saxy AFAICT.
The text was updated successfully, but these errors were encountered: