Check engine reference for null before flushing

If we close the shard before the engine is started we see a NPE in
the logs which is not problematic since the relevant parts are in a
finally block. Yet, the NPE is unnecessary and can be confusing.

Author: s1monw

src/main/java/org/elasticsearch/index/shard/IndexShard.java

@@ -756,7 +756,7 @@ public void close(String reason, boolean flushEngine) throws IOException {
             } finally {
                 final Engine engine = this.currentEngineReference.getAndSet(null);
                 try {
-                    if (flushEngine && this.flushOnClose) {
+                    if (engine != null && flushEngine && this.flushOnClose) {
                         engine.flushAndClose();
                     }
                 } finally {