Revert detach of context when running on event-loop (we lose a bit of performance)

Author: jknack

jooby/src/main/java/io/jooby/internal/Pipeline.java

@@ -133,9 +133,6 @@ public static Handler compute(ClassLoader loader, Route route, ExecutionMode mod
     }
     /** Context: */
     if (Context.class.isAssignableFrom(type)) {
-      if (executor == null && mode == ExecutionMode.EVENT_LOOP) {
-        return next(mode, executor, new DetachHandler(decorate(route)), false);
-      }
       return next(mode, executor, decorate(route, new SendDirect(route.getPipeline())), true);
     }
     /** InputStream: */
@@ -176,10 +173,6 @@ public static Handler compute(ClassLoader loader, Route route, ExecutionMode mod
     return next(mode, executor, decorate(route, new DefaultHandler(route.getPipeline())), true);
   }
 
-  private static Handler decorate(Route route) {
-    return decorate(route, route.getPipeline());
-  }
-
   private static Handler decorate(Route route, Handler handler) {
     Handler pipeline = handler;
     if (route.isHttpHead()) {

tests/src/test/java/io/jooby/FeaturedTest.java

@@ -2339,7 +2339,8 @@ public void accessToFlashScopeFromTemplateEngine(ServerTestRunner runner) {
     });
   }
 
-  @ServerTest(executionMode = EVENT_LOOP)
+  // TODO: We might add this back but using a different implementation:
+  //  @ServerTest(executionMode = EVENT_LOOP)
   @DisplayName("Context detaches when running in event-loop and returns a Context")
   public void detachOnEventLoop(ServerTestRunner runner) {
     runner.define(app -> {