pmouawad commented on pull request #615:
URL: https://github.com/apache/jmeter/pull/615#issuecomment-705181585


   Hello @delirius325 , 
   The drawbacks I see currently are:
   
   - For signing on AWS ElasticSearch, It introduces a dependency on 
aws-java-sdk and google guava which might not be acceptable. This should be 
available as a pluggable thing instead
   -  We need to maintain in core a dependency on a 3rd party (Elastic Search) 
that has a faster pace in terms of releases
   - is there a reason for not keeping the plugin as is and contributing it to 
JMeter core ? 
   
   The addition looks really useful, but I am wondering if it's not better that 
this one remains as a plugin, as its pace of release can be faster than JMeter 
one.
   
   What's your thoughts ? 
   Regards


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to