The spotless should take care of unified imports ordering (libs/ subp…

…rojects) (opensearch-project#9309)

Signed-off-by: Owais Kazi <owaiskazi19@gmail.com>

libs/common/src/main/java/org/opensearch/common/Nullable.java

@@ -34,6 +34,7 @@
 
 import javax.annotation.CheckForNull;
 import javax.annotation.meta.TypeQualifierNickname;
+
 import java.lang.annotation.Documented;
 import java.lang.annotation.ElementType;
 import java.lang.annotation.Retention;

libs/core/src/main/java/org/opensearch/ExceptionsHelper.java

@@ -33,15 +33,16 @@
 package org.opensearch;
 
 import com.fasterxml.jackson.core.JsonParseException;
+
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
 import org.apache.lucene.index.CorruptIndexException;
 import org.apache.lucene.index.IndexFormatTooNewException;
 import org.apache.lucene.index.IndexFormatTooOldException;
-import org.opensearch.core.action.ShardOperationFailedException;
 import org.opensearch.common.CheckedRunnable;
 import org.opensearch.common.CheckedSupplier;
 import org.opensearch.common.Nullable;
+import org.opensearch.core.action.ShardOperationFailedException;
 import org.opensearch.core.common.compress.NotXContentException;
 import org.opensearch.core.concurrency.OpenSearchRejectedExecutionException;
 import org.opensearch.core.index.Index;

libs/core/src/main/java/org/opensearch/OpenSearchException.java

@@ -34,20 +34,20 @@
 import org.opensearch.common.CheckedFunction;
 import org.opensearch.common.Nullable;
 import org.opensearch.common.collect.Tuple;
+import org.opensearch.core.ParseField;
+import org.opensearch.core.common.Strings;
 import org.opensearch.core.common.io.stream.StreamInput;
 import org.opensearch.core.common.io.stream.StreamOutput;
 import org.opensearch.core.common.io.stream.Writeable;
-import org.opensearch.core.ParseField;
-import org.opensearch.core.common.Strings;
 import org.opensearch.core.common.logging.LoggerMessageFormat;
+import org.opensearch.core.index.Index;
+import org.opensearch.core.index.shard.ShardId;
+import org.opensearch.core.rest.RestStatus;
 import org.opensearch.core.xcontent.ToXContent;
 import org.opensearch.core.xcontent.ToXContentFragment;
 import org.opensearch.core.xcontent.XContentBuilder;
 import org.opensearch.core.xcontent.XContentParseException;
 import org.opensearch.core.xcontent.XContentParser;
-import org.opensearch.core.index.Index;
-import org.opensearch.core.index.shard.ShardId;
-import org.opensearch.core.rest.RestStatus;
 
 import java.io.IOException;
 import java.util.ArrayList;
@@ -61,12 +61,11 @@
 import java.util.concurrent.ConcurrentHashMap;
 
 import static java.util.Collections.emptyMap;
+import static java.util.Collections.singletonMap;
 import static org.opensearch.OpenSearchException.OpenSearchExceptionHandleRegistry.registerExceptionHandle;
 import static org.opensearch.core.xcontent.XContentParserUtils.ensureExpectedToken;
 import static org.opensearch.core.xcontent.XContentParserUtils.ensureFieldName;
 
-import static java.util.Collections.singletonMap;
-
 /**
  * A core library base class for all opensearch exceptions.
  *

libs/core/src/main/java/org/opensearch/core/action/ShardOperationFailedException.java

@@ -34,8 +34,8 @@
 
 import org.opensearch.common.Nullable;
 import org.opensearch.core.common.io.stream.Writeable;
-import org.opensearch.core.xcontent.ToXContentObject;
 import org.opensearch.core.rest.RestStatus;
+import org.opensearch.core.xcontent.ToXContentObject;
 
 import java.util.Objects;
 

libs/core/src/main/java/org/opensearch/core/action/support/DefaultShardOperationFailedException.java

@@ -34,15 +34,15 @@
 
 import org.opensearch.ExceptionsHelper;
 import org.opensearch.OpenSearchException;
-import org.opensearch.core.action.ShardOperationFailedException;
 import org.opensearch.core.ParseField;
+import org.opensearch.core.action.ShardOperationFailedException;
 import org.opensearch.core.common.io.stream.StreamInput;
 import org.opensearch.core.common.io.stream.StreamOutput;
 import org.opensearch.core.common.io.stream.Writeable;
+import org.opensearch.core.rest.RestStatus;
 import org.opensearch.core.xcontent.ConstructingObjectParser;
 import org.opensearch.core.xcontent.XContentBuilder;
 import org.opensearch.core.xcontent.XContentParser;
-import org.opensearch.core.rest.RestStatus;
 
 import java.io.IOException;
 

libs/core/src/main/java/org/opensearch/core/common/ParsingException.java

@@ -35,10 +35,10 @@
 import org.opensearch.OpenSearchException;
 import org.opensearch.core.common.io.stream.StreamInput;
 import org.opensearch.core.common.io.stream.StreamOutput;
+import org.opensearch.core.rest.RestStatus;
 import org.opensearch.core.xcontent.XContentBuilder;
 import org.opensearch.core.xcontent.XContentLocation;
 import org.opensearch.core.xcontent.XContentParser;
-import org.opensearch.core.rest.RestStatus;
 
 import java.io.IOException;
 

libs/core/src/main/java/org/opensearch/core/common/breaker/CircuitBreakingException.java

@@ -34,8 +34,8 @@
 import org.opensearch.OpenSearchException;
 import org.opensearch.core.common.io.stream.StreamInput;
 import org.opensearch.core.common.io.stream.StreamOutput;
-import org.opensearch.core.xcontent.XContentBuilder;
 import org.opensearch.core.rest.RestStatus;
+import org.opensearch.core.xcontent.XContentBuilder;
 
 import java.io.IOException;
 

libs/core/src/main/java/org/opensearch/core/common/io/stream/StreamInput.java

@@ -46,12 +46,12 @@
 import org.opensearch.Version;
 import org.opensearch.common.CharArrays;
 import org.opensearch.common.Nullable;
+import org.opensearch.common.unit.TimeValue;
+import org.opensearch.core.common.Strings;
 import org.opensearch.core.common.bytes.BytesArray;
 import org.opensearch.core.common.bytes.BytesReference;
 import org.opensearch.core.common.settings.SecureString;
 import org.opensearch.core.common.text.Text;
-import org.opensearch.common.unit.TimeValue;
-import org.opensearch.core.common.Strings;
 import org.opensearch.core.concurrency.OpenSearchRejectedExecutionException;
 import org.opensearch.core.xcontent.MediaType;
 import org.opensearch.core.xcontent.MediaTypeRegistry;

libs/core/src/main/java/org/opensearch/core/common/io/stream/StreamOutput.java

@@ -45,13 +45,13 @@
 import org.opensearch.Version;
 import org.opensearch.common.CharArrays;
 import org.opensearch.common.Nullable;
+import org.opensearch.common.unit.TimeValue;
 import org.opensearch.core.common.bytes.BytesArray;
 import org.opensearch.core.common.bytes.BytesReference;
 import org.opensearch.core.common.io.stream.Writeable.WriteableRegistry;
 import org.opensearch.core.common.io.stream.Writeable.Writer;
 import org.opensearch.core.common.settings.SecureString;
 import org.opensearch.core.common.text.Text;
-import org.opensearch.common.unit.TimeValue;
 import org.opensearch.core.concurrency.OpenSearchRejectedExecutionException;
 
 import java.io.EOFException;

libs/core/src/main/java/org/opensearch/core/index/Index.java

@@ -33,10 +33,10 @@
 package org.opensearch.core.index;
 
 import org.opensearch.core.ParseField;
+import org.opensearch.core.common.Strings;
 import org.opensearch.core.common.io.stream.StreamInput;
 import org.opensearch.core.common.io.stream.StreamOutput;
 import org.opensearch.core.common.io.stream.Writeable;
-import org.opensearch.core.common.Strings;
 import org.opensearch.core.xcontent.ObjectParser;
 import org.opensearch.core.xcontent.ToXContentObject;
 import org.opensearch.core.xcontent.XContentBuilder;

libs/core/src/main/java/org/opensearch/core/index/shard/ShardId.java

@@ -36,9 +36,9 @@
 import org.opensearch.core.common.io.stream.StreamInput;
 import org.opensearch.core.common.io.stream.StreamOutput;
 import org.opensearch.core.common.io.stream.Writeable;
+import org.opensearch.core.index.Index;
 import org.opensearch.core.xcontent.ToXContentFragment;
 import org.opensearch.core.xcontent.XContentBuilder;
-import org.opensearch.core.index.Index;
 
 import java.io.IOException;
 

libs/core/src/main/java/org/opensearch/core/tasks/resourcetracker/TaskResourceUsage.java

@@ -8,11 +8,11 @@
 
 package org.opensearch.core.tasks.resourcetracker;
 
+import org.opensearch.core.ParseField;
 import org.opensearch.core.common.Strings;
 import org.opensearch.core.common.io.stream.StreamInput;
 import org.opensearch.core.common.io.stream.StreamOutput;
 import org.opensearch.core.common.io.stream.Writeable;
-import org.opensearch.core.ParseField;
 import org.opensearch.core.xcontent.ConstructingObjectParser;
 import org.opensearch.core.xcontent.MediaTypeRegistry;
 import org.opensearch.core.xcontent.ToXContentFragment;

libs/core/src/main/java/org/opensearch/core/tasks/resourcetracker/TaskThreadUsage.java

@@ -8,11 +8,11 @@
 
 package org.opensearch.core.tasks.resourcetracker;
 
+import org.opensearch.core.ParseField;
 import org.opensearch.core.common.Strings;
 import org.opensearch.core.common.io.stream.StreamInput;
 import org.opensearch.core.common.io.stream.StreamOutput;
 import org.opensearch.core.common.io.stream.Writeable;
-import org.opensearch.core.ParseField;
 import org.opensearch.core.xcontent.ConstructingObjectParser;
 import org.opensearch.core.xcontent.MediaTypeRegistry;
 import org.opensearch.core.xcontent.ToXContentFragment;

libs/core/src/main/java/org/opensearch/core/util/FileSystemUtils.java

@@ -34,8 +34,8 @@
 
 import org.apache.logging.log4j.Logger;
 import org.apache.lucene.util.Constants;
-import org.opensearch.common.util.io.IOUtils;
 import org.opensearch.common.SuppressForbidden;
+import org.opensearch.common.util.io.IOUtils;
 
 import java.io.IOException;
 import java.io.InputStream;

libs/core/src/main/java/org/opensearch/core/xcontent/MediaTypeRegistry.java

@@ -40,7 +40,6 @@
 import java.io.InputStream;
 import java.io.OutputStream;
 import java.io.UncheckedIOException;
-
 import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.List;

libs/core/src/main/java/org/opensearch/core/xcontent/XContentParserUtils.java

@@ -33,8 +33,8 @@
 package org.opensearch.core.xcontent;
 
 import org.opensearch.core.common.ParsingException;
-import org.opensearch.core.common.bytes.BytesArray;
 import org.opensearch.core.common.Strings;
+import org.opensearch.core.common.bytes.BytesArray;
 import org.opensearch.core.xcontent.XContentParser.Token;
 
 import java.io.IOException;

libs/core/src/test/java/org/opensearch/core/action/support/DefaultShardOperationFailedExceptionTests.java

@@ -45,13 +45,13 @@
 import org.opensearch.core.common.bytes.BytesReference;
 import org.opensearch.core.common.io.stream.StreamInput;
 import org.opensearch.core.concurrency.OpenSearchRejectedExecutionException;
+import org.opensearch.core.index.Index;
+import org.opensearch.core.index.shard.ShardId;
+import org.opensearch.core.rest.RestStatus;
 import org.opensearch.core.xcontent.MediaTypeRegistry;
 import org.opensearch.core.xcontent.XContent;
 import org.opensearch.core.xcontent.XContentBuilder;
 import org.opensearch.core.xcontent.XContentParser;
-import org.opensearch.core.index.Index;
-import org.opensearch.core.index.shard.ShardId;
-import org.opensearch.core.rest.RestStatus;
 import org.opensearch.test.OpenSearchTestCase;
 
 import java.io.EOFException;

libs/core/src/test/java/org/opensearch/core/common/io/stream/BaseStreamTests.java

@@ -36,10 +36,10 @@
 import org.opensearch.common.CheckedBiConsumer;
 import org.opensearch.common.CheckedConsumer;
 import org.opensearch.common.CheckedFunction;
+import org.opensearch.common.collect.Tuple;
 import org.opensearch.common.io.stream.BytesStreamOutput;
 import org.opensearch.core.common.bytes.BytesArray;
 import org.opensearch.core.common.bytes.BytesReference;
-import org.opensearch.common.collect.Tuple;
 import org.opensearch.core.common.settings.SecureString;
 import org.opensearch.script.JodaCompatibleZonedDateTime;
 import org.opensearch.test.OpenSearchTestCase;

libs/core/src/test/java/org/opensearch/core/common/io/stream/NamedWriteableRegistryTests.java

@@ -32,12 +32,12 @@
 
 package org.opensearch.core.common.io.stream;
 
+import org.opensearch.test.OpenSearchTestCase;
+
 import java.io.IOException;
 import java.util.Arrays;
 import java.util.Collections;
 
-import org.opensearch.test.OpenSearchTestCase;
-
 public class NamedWriteableRegistryTests extends OpenSearchTestCase {
 
     private static class DummyNamedWriteable implements NamedWriteable {

libs/core/src/test/java/org/opensearch/core/internal/net/NetUtilsTests.java

@@ -33,8 +33,8 @@
 package org.opensearch.core.internal.net;
 
 import org.apache.lucene.util.Constants;
-import org.opensearch.common.util.net.NetUtils;
 import org.opensearch.common.util.io.IOUtils;
+import org.opensearch.common.util.net.NetUtils;
 import org.opensearch.test.OpenSearchTestCase;
 
 import java.lang.Runtime.Version;

libs/core/src/test/java/org/opensearch/core/util/FileSystemUtilsTests.java

@@ -32,8 +32,8 @@
 
 package org.opensearch.core.util;
 
-import org.apache.lucene.util.Constants;
 import org.apache.lucene.tests.util.LuceneTestCase.SuppressFileSystems;
+import org.apache.lucene.util.Constants;
 import org.opensearch.common.io.PathUtils;
 import org.opensearch.test.OpenSearchTestCase;
 import org.junit.Before;

libs/dissect/src/test/java/org/opensearch/dissect/DissectParserTests.java

@@ -34,17 +34,19 @@
 
 import com.fasterxml.jackson.databind.JsonNode;
 import com.fasterxml.jackson.databind.ObjectMapper;
+
 import org.opensearch.test.OpenSearchTestCase;
 import org.hamcrest.CoreMatchers;
 import org.hamcrest.Matchers;
-import org.mockito.internal.util.collections.Sets;
 
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Iterator;
 import java.util.List;
 import java.util.Map;
 
+import org.mockito.internal.util.collections.Sets;
+
 import static com.carrotsearch.randomizedtesting.RandomizedTest.randomAsciiAlphanumOfLengthBetween;
 
 public class DissectParserTests extends OpenSearchTestCase {

libs/grok/src/main/java/org/opensearch/grok/Grok.java

@@ -32,15 +32,6 @@
 
 package org.opensearch.grok;
 
-import org.jcodings.specific.UTF8Encoding;
-import org.joni.Matcher;
-import org.joni.NameEntry;
-import org.joni.Option;
-import org.joni.Regex;
-import org.joni.Region;
-import org.joni.Syntax;
-import org.joni.exception.ValueException;
-
 import java.io.BufferedReader;
 import java.io.IOException;
 import java.io.InputStream;
@@ -56,6 +47,15 @@
 import java.util.Stack;
 import java.util.function.Consumer;
 
+import org.jcodings.specific.UTF8Encoding;
+import org.joni.Matcher;
+import org.joni.NameEntry;
+import org.joni.Option;
+import org.joni.Regex;
+import org.joni.Region;
+import org.joni.Syntax;
+import org.joni.exception.ValueException;
+
 import static java.util.Collections.unmodifiableList;
 
 public final class Grok {

libs/grok/src/main/java/org/opensearch/grok/GrokCaptureConfig.java

@@ -32,15 +32,15 @@
 
 package org.opensearch.grok;
 
-import org.joni.NameEntry;
-
 import java.nio.charset.StandardCharsets;
 import java.util.function.Consumer;
 import java.util.function.DoubleConsumer;
 import java.util.function.Function;
 import java.util.function.IntConsumer;
 import java.util.function.LongConsumer;
 
+import org.joni.NameEntry;
+
 /**
  * Configuration for a value that {@link Grok} can capture.
  */

libs/grok/src/main/java/org/opensearch/grok/GrokCaptureExtracter.java

@@ -32,13 +32,13 @@
 
 package org.opensearch.grok;
 
-import org.joni.Region;
-
 import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
 
+import org.joni.Region;
+
 import static java.util.Collections.emptyMap;
 
 /**

libs/grok/src/main/java/org/opensearch/grok/GrokCaptureType.java

@@ -33,11 +33,12 @@
 package org.opensearch.grok;
 
 import org.opensearch.grok.GrokCaptureConfig.NativeExtracterMap;
-import org.joni.Region;
 
 import java.nio.charset.StandardCharsets;
 import java.util.function.Consumer;
 
+import org.joni.Region;
+
 /**
  * The type defined for the field in the pattern.
  */