Merge pull request #447 from mattrjacobs/fix-sync-queue

Fixed HystrixThreadPool to always return true for isQueueSpaceAvailable() when size = 0 (the SynchronousQueue case)

hystrix-core/src/main/java/com/netflix/hystrix/HystrixThreadPool.java

@@ -202,11 +202,11 @@ public void markThreadCompletion() {
         /**
          * Whether the threadpool queue has space available according to the <code>queueSizeRejectionThreshold</code> settings.
          * <p>
-         * If a SynchronousQueue implementation is used (<code>maxQueueSize</code> == -1), it always returns 0 as the size so this would always return true.
+         * If a SynchronousQueue implementation is used (<code>maxQueueSize</code> <= 0), it always returns 0 as the size so this would always return true.
          */
         @Override
         public boolean isQueueSpaceAvailable() {
-            if (properties.maxQueueSize().get() < 0) {
+            if (properties.maxQueueSize().get() <= 0) {
                 // we don't have a queue so we won't look for space but instead
                 // let the thread-pool reject or not
                 return true;

hystrix-core/src/main/java/com/netflix/hystrix/strategy/concurrency/HystrixContextScheduler.java

@@ -180,7 +180,6 @@ public void run() {
 
         @Override
         public Subscription schedule(Action0 action, long delayTime, TimeUnit unit) {
-            System.out.println("delayed scheduling");
             throw new IllegalStateException("Hystrix does not support delayed scheduling");
         }