remove/cleanup/etc

extensions-core/multi-stage-query/src/test/quidem/org.apache.druid.msq.quidem.MSQQuidemTest/msq2.iq

@@ -1,32 +0,0 @@
-!set isNative false
-!set isMSQ true
-!set plannerStrategy DECOUPLED
-!if (isMSQ) {
-!use druidtest://?componentSupplier=DrillWindowQueryMSQComponentSupplier
-!}
-!if (isNative) {
-!use druidtest://?numMergeBuffers=3
-!}
-
-!set outputformat mysql
-
-select 2;
-+--------+
-| EXPR$0 |
-+--------+
-|      2 |
-+--------+
-(1 row)
-
-!ok
-
-
-!msqPlan
-
-!if (isMSQ) {
-!msqPlan
-!}
-
-!if (isNative) {
-!msqPlan
-!}

quidem-ut/src/test/java/org/apache/druid/quidem/LauncherSmokeTest.java

@@ -69,8 +69,8 @@ public class LauncherSmokeTest
     HttpGet request = new HttpGet("http://localhost:12345/status");
     request.addHeader("Content-Type", "application/json");
     CloseableHttpResponse response = client.execute(request);
+    assertEquals(200, response.getStatusLine().getStatusCode());
     String responseStr = EntityUtils.toString(response.getEntity());
     MatcherAssert.assertThat(responseStr, Matchers.containsString("\"version\":\""));
-    assertEquals(200, response.getStatusLine().getStatusCode());
   }
 }

server/src/main/java/org/apache/druid/server/security/AuthorizationUtils.java

@@ -218,7 +218,7 @@ public class AuthorizationUtils
     }
 
     if (request.getAttribute(AuthConfig.DRUID_AUTHORIZATION_CHECKED) != null) {
-      //throw new ISE("Request already had authorization check.");
+      throw new ISE("Request already had authorization check.");
     }
 
     Access access = authorizeAllResourceActions(