Havret commented on PR #92: URL: https://github.com/apache/activemq-nms-amqp/pull/92#issuecomment-3041341644
@thompson-tomo The only dependency we could potentially remove (or make conditional with a compiler flag) is System.Threading.Tasks.Dataflow. However, I'm not sure it's worth the overhead of maintaining a specific .NET version for this. We're not using any APIs outside of .NET Standard 2.0. If there's a need to use APIs available only in newer TFMs, I'm open to adding those alongside .NET Standard. But doing so just for the sake of it doesn’t make much sense, since any performance improvements come from the runtime, not the standard itself. So for now, I’m not in favor of proceeding with this change. -- 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: gitbox-unsubscr...@activemq.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@activemq.apache.org For additional commands, e-mail: gitbox-h...@activemq.apache.org For further information, visit: https://activemq.apache.org/contact
