[GitHub] [accumulo] ctubbsii commented on issue #1532: Class not found org/apache/tools/ant/taskdefs/Execute - used by commons-configuration

[GitBox <gi...@apache.org>]

ctubbsii commented on issue #1532: Class not found org/apache/tools/ant/taskdefs/Execute - used by commons-configuration
URL: https://github.com/apache/accumulo/issues/1532#issuecomment-592049874
 
 
   I see. Since we've already dropped support for the legacy metrics in 2.x and upgraded to commons-configuration2, it kind of feels like this backport might be more trouble than it's worth... causing us to revisit already solved issues.
   
   If backporting these GC metrics to 1.x is going to cause significant problems that can't easily be worked around, I'm inclined to just revert those changes in 1.x and make people wait until they upgrade to get the new GC metrics if they need them.
   
   If it's an easy fix, though... without breaking the legacy metrics in 1.x, that's preferred, of course (best of both worlds, and all that). Is it possible to just throw ant on the class path ourselves (as optional) to fix the problem? And, if that works, we can just tell people they might need ant on their class path to get the feature in 1.x.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services