Disable security manager in Nailgun

The security manager installed by Nailgun seems to add a noticeable
overhead in IO operations.  facebookarchive#134

As a result, we don't install a security manager. Nailgun only uses it
to trap exit, and in our case we don't need to do that since Bloop
doesn't define any `nailShutdown`.

nailgun-server/src/main/java/com/martiansoftware/nailgun/NGServer.java

@@ -17,6 +17,9 @@
  */
 package com.martiansoftware.nailgun;
 
+import com.martiansoftware.nailgun.builtins.DefaultNail;
+import com.sun.jna.Platform;
+
 import java.io.InputStream;
 import java.io.PrintStream;
 import java.net.InetAddress;
@@ -27,9 +30,6 @@
 import java.util.Map;
 import java.util.concurrent.atomic.AtomicBoolean;
 
-import com.martiansoftware.nailgun.builtins.DefaultNail;
-import com.sun.jna.Platform;
-
 /**
  * <p>Listens for new connections from NailGun clients and launches NGSession
  * threads to process them.</p>
@@ -329,7 +329,8 @@ public void shutdown(boolean exitVM) {
         System.setOut(out);
         System.setErr(err);
 
-        System.setSecurityManager(originalSecurityManager);
+        // Security manager is always disabled
+        //System.setSecurityManager(originalSecurityManager);
 
         if (exitVM) {
             System.exit(0);
@@ -363,10 +364,11 @@ public void run() {
         NGSession sessionOnDeck = null;
 
         originalSecurityManager = System.getSecurityManager();
-        System.setSecurityManager(
-                new NGSecurityManager(
-                originalSecurityManager));
 
+        // Remove overhead of security manager https://github.com/facebook/nailgun/issues/134
+        //System.setSecurityManager(
+        //        new NGSecurityManager(
+        //        originalSecurityManager));
 
         synchronized (System.in) {
             if (!(System.in instanceof ThreadLocalInputStream)) {