Corrected bounds.

Author: RamSaw

core/src/main/java/org/jetbrains/research/groups/ml_methods/algorithm/HAC.java

@@ -27,9 +27,9 @@
 
 public class HAC extends AbstractAlgorithm {
     private static final Logger LOGGER = Logging.getLogger(HAC.class);
-    private static final int MAX_NUMBER_OF_CLASSES = 3000;
-    private static final int MAX_NUMBER_OF_METHODS = 20000;
-    private static final int MAX_NUMBER_OF_FIELDS = 9000;
+    private static final int MAX_NUMBER_OF_CLASSES = 1000;
+    private static final int MAX_NUMBER_OF_METHODS = 3000;
+    private static final int MAX_NUMBER_OF_FIELDS = 1500;
     private static final double ACCURACY = 1;
 
     private static final @NotNull DistanceCalculator distanceCalculator = RelevanceBasedDistanceCalculator.getInstance();
@@ -51,16 +51,20 @@ public HAC() {
     @NotNull
     @Override
     public AlgorithmResult execute(@NotNull AttributesStorage attributes, @Nullable ExecutorService service, boolean enableFieldRefactorings) {
-        if (attributes.getClassesAttributes().size() > MAX_NUMBER_OF_CLASSES ||
-                attributes.getMethodsAttributes().size() > MAX_NUMBER_OF_METHODS ||
-                attributes.getFieldsAttributes().size() > MAX_NUMBER_OF_FIELDS) {
+        if (willHaveLongTimeExecution(attributes)) {
             LOGGER.warn("HAC haven't been executed because project is too large and HAC will work too long");
             return new AlgorithmResult(AlgorithmType.HAC,
                     new TooLargeProjectException("HAC execution will be too long on this project"));
         }
         return super.execute(attributes, service, enableFieldRefactorings);
     }
 
+    private boolean willHaveLongTimeExecution(@NotNull AttributesStorage attributes) {
+        return attributes.getClassesAttributes().size() > MAX_NUMBER_OF_CLASSES ||
+                attributes.getMethodsAttributes().size() > MAX_NUMBER_OF_METHODS ||
+                attributes.getFieldsAttributes().size() > MAX_NUMBER_OF_FIELDS;
+    }
+
     private static class HACExecutor implements Executor {
         private final SortedSet<Triple> heap = new TreeSet<>();
         private final Map<Long, Triple> triples = new HashMap<>();