Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Spotless] Fixes license headers (PR 3) #308

Closed
wants to merge 2 commits into from
Closed

[Spotless] Fixes license headers (PR 3) #308

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
edfe628
Add remaining header checks.
MitchellGale Jul 21, 2023
5d27c9d
Updated spotless build gradle to include license header update.
MitchellGale Jul 21, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions ...mh/java/org/opensearch/sql/expression/operator/predicate/ComparisonOperatorBenchmark.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.expression.operator.predicate;

import static org.opensearch.sql.data.model.ExprValueUtils.fromObjectValue;
Expand Down
8 changes: 4 additions & 4 deletions build.gradle
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -88,10 +88,10 @@ spotless {
exclude '**/build/**', '**/build-*/**'
}
importOrder()
// licenseHeader("/*\n" +
// " * Copyright OpenSearch Contributors\n" +
// " * SPDX-License-Identifier: Apache-2.0\n" +
// " */\n\n\n")
licenseHeader("/*\n" +
" * Copyright OpenSearch Contributors\n" +
" * SPDX-License-Identifier: Apache-2.0\n" +
" */\n\n\n")
removeUnusedImports()
trimTrailingWhitespace()
endWithNewline()
Expand Down
6 changes: 6 additions & 0 deletions common/src/main/java/org/opensearch/sql/common/antlr/Parser.java
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,3 +1,9 @@
/*
* Copyright OpenSearch Contributors
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.common.antlr;

import org.antlr.v4.runtime.tree.ParseTree;
Expand Down
7 changes: 3 additions & 4 deletions common/src/main/java/org/opensearch/sql/common/authinterceptors/AwsSigningInterceptor.java
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,10 +1,9 @@
/*
*
* * Copyright OpenSearch Contributors
* * SPDX-License-Identifier: Apache-2.0
*
* Copyright OpenSearch Contributors
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.common.authinterceptors;

import com.amazonaws.auth.AWSCredentials;
Expand Down
7 changes: 3 additions & 4 deletions .../main/java/org/opensearch/sql/common/authinterceptors/BasicAuthenticationInterceptor.java
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,10 +1,9 @@
/*
*
* * Copyright OpenSearch Contributors
* * SPDX-License-Identifier: Apache-2.0
*
* Copyright OpenSearch Contributors
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.common.authinterceptors;

import java.io.IOException;
Expand Down
1 change: 1 addition & 0 deletions common/src/main/java/org/opensearch/sql/common/grok/Converter.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.common.grok;

import java.time.Instant;
Expand Down
1 change: 1 addition & 0 deletions common/src/main/java/org/opensearch/sql/common/grok/Grok.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.common.grok;

import java.io.Serializable;
Expand Down
1 change: 1 addition & 0 deletions common/src/main/java/org/opensearch/sql/common/grok/GrokCompiler.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.common.grok;

import static java.lang.String.format;
Expand Down
1 change: 1 addition & 0 deletions common/src/main/java/org/opensearch/sql/common/grok/GrokUtils.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.common.grok;

import java.util.LinkedHashMap;
Expand Down
1 change: 1 addition & 0 deletions common/src/main/java/org/opensearch/sql/common/grok/Match.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.common.grok;


Expand Down
1 change: 1 addition & 0 deletions common/src/main/java/org/opensearch/sql/common/grok/exception/GrokException.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.common.grok.exception;

/**
Expand Down
7 changes: 3 additions & 4 deletions ...n/src/test/java/org/opensearch/sql/common/authinterceptors/AwsSigningInterceptorTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,10 +1,9 @@
/*
*
* * Copyright OpenSearch Contributors
* * SPDX-License-Identifier: Apache-2.0
*
* Copyright OpenSearch Contributors
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.common.authinterceptors;


Expand Down
7 changes: 3 additions & 4 deletions ...t/java/org/opensearch/sql/common/authinterceptors/BasicAuthenticationInterceptorTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,10 +1,9 @@
/*
*
* * Copyright OpenSearch Contributors
* * SPDX-License-Identifier: Apache-2.0
*
* Copyright OpenSearch Contributors
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.common.authinterceptors;

import java.util.Collections;
Expand Down
1 change: 1 addition & 0 deletions common/src/test/java/org/opensearch/sql/common/grok/ApacheDataTypeTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.common.grok;


Expand Down
1 change: 1 addition & 0 deletions common/src/test/java/org/opensearch/sql/common/grok/ApacheTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.common.grok;


Expand Down
1 change: 1 addition & 0 deletions common/src/test/java/org/opensearch/sql/common/grok/BasicTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.common.grok;

import static junit.framework.TestCase.assertTrue;
Expand Down
1 change: 1 addition & 0 deletions common/src/test/java/org/opensearch/sql/common/grok/CaptureTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.common.grok;

import static org.hamcrest.CoreMatchers.containsString;
Expand Down
1 change: 1 addition & 0 deletions common/src/test/java/org/opensearch/sql/common/grok/GrokDocumentationTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.common.grok;

import static org.hamcrest.Matchers.containsInAnyOrder;
Expand Down
1 change: 1 addition & 0 deletions common/src/test/java/org/opensearch/sql/common/grok/GrokListTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.common.grok;

import static org.junit.Assert.assertEquals;
Expand Down
1 change: 1 addition & 0 deletions common/src/test/java/org/opensearch/sql/common/grok/GrokTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.common.grok;

import static java.lang.String.format;
Expand Down
1 change: 1 addition & 0 deletions common/src/test/java/org/opensearch/sql/common/grok/MessagesTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.common.grok;

import static org.junit.Assert.assertNotNull;
Expand Down
1 change: 1 addition & 0 deletions common/src/test/java/org/opensearch/sql/common/grok/ResourceManager.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.common.grok;

/**
Expand Down
1 change: 0 additions & 1 deletion legacy/src/main/java/org/opensearch/sql/legacy/executor/cursor/CursorRestExecutor.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
*/



package org.opensearch.sql.legacy.executor.cursor;

import java.util.Map;
Expand Down
1 change: 1 addition & 0 deletions .../src/test/java/org/opensearch/sql/legacy/plugin/RestSQLQueryActionCursorFallbackTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.legacy.plugin;

import static org.junit.Assert.assertFalse;
Expand Down
1 change: 1 addition & 0 deletions ...t/java/org/opensearch/sql/legacy/query/planner/physical/node/scroll/SearchHitRowTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.legacy.query.planner.physical.node.scroll;

import static org.junit.Assert.assertEquals;
Expand Down
6 changes: 6 additions & 0 deletions opensearch/src/main/java/org/opensearch/sql/opensearch/client/MLClient.java
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,3 +1,9 @@
/*
* Copyright OpenSearch Contributors
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.opensearch.client;

import org.opensearch.client.node.NodeClient;
Expand Down
1 change: 1 addition & 0 deletions opensearch/src/main/java/org/opensearch/sql/opensearch/data/type/OpenSearchDateType.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.opensearch.data.type;

import static org.opensearch.common.time.DateFormatter.splitCombinedPatterns;
Expand Down
1 change: 1 addition & 0 deletions opensearch/src/main/java/org/opensearch/sql/opensearch/data/type/OpenSearchTextType.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.opensearch.data.type;

import static org.opensearch.sql.data.type.ExprCoreType.STRING;
Expand Down
6 changes: 2 additions & 4 deletions opensearch/src/main/java/org/opensearch/sql/opensearch/executor/OpenSearchQueryManager.java
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,11 +1,9 @@
/*
* Copyright OpenSearch Contributors
* SPDX-License-Identifier: Apache-2.0
*
* The OpenSearch Contributors require contributions made to
* this file be licensed under the Apache-2.0 license or a
* compatible open source license.
*/


package org.opensearch.sql.opensearch.executor;

import java.util.Map;
Expand Down
1 change: 1 addition & 0 deletions ...earch/src/main/java/org/opensearch/sql/opensearch/planner/physical/MLCommonsOperator.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.opensearch.planner.physical;

import static org.opensearch.ml.common.FunctionName.KMEANS;
Expand Down
1 change: 1 addition & 0 deletions opensearch/src/main/java/org/opensearch/sql/opensearch/planner/physical/MLOperator.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.opensearch.planner.physical;

import java.util.ArrayList;
Expand Down
13 changes: 3 additions & 10 deletions .../src/main/java/org/opensearch/sql/opensearch/response/agg/CompositeAggregationParser.java
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,16 +1,9 @@
/*
* Licensed under the Apache License, Version 2.0 (the "License").
* You may not use this file except in compliance with the License.
* A copy of the License is located at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* or in the "license" file accompanying this file. This file is distributed
* on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either
* express or implied. See the License for the specific language governing
* permissions and limitations under the License.
* Copyright OpenSearch Contributors
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.opensearch.response.agg;

import java.util.Arrays;
Expand Down
13 changes: 3 additions & 10 deletions opensearch/src/main/java/org/opensearch/sql/opensearch/response/agg/FilterParser.java
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,16 +1,9 @@
/*
* Licensed under the Apache License, Version 2.0 (the "License").
* You may not use this file except in compliance with the License.
* A copy of the License is located at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* or in the "license" file accompanying this file. This file is distributed
* on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either
* express or implied. See the License for the specific language governing
* permissions and limitations under the License.
* Copyright OpenSearch Contributors
* SPDX-License-Identifier: Apache-2.0
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like this PR has quite a few spots with the same header that Yury has mentioned here and should probably should be looked into as well

*/


package org.opensearch.sql.opensearch.response.agg;

import java.util.Map;
Expand Down
13 changes: 3 additions & 10 deletions opensearch/src/main/java/org/opensearch/sql/opensearch/response/agg/MetricParser.java
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,16 +1,9 @@
/*
* Licensed under the Apache License, Version 2.0 (the "License").
* You may not use this file except in compliance with the License.
* A copy of the License is located at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* or in the "license" file accompanying this file. This file is distributed
* on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either
* express or implied. See the License for the specific language governing
* permissions and limitations under the License.
* Copyright OpenSearch Contributors
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.opensearch.response.agg;

import java.util.Map;
Expand Down
13 changes: 3 additions & 10 deletions opensearch/src/main/java/org/opensearch/sql/opensearch/response/agg/MetricParserHelper.java
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,16 +1,9 @@
/*
* Licensed under the Apache License, Version 2.0 (the "License").
* You may not use this file except in compliance with the License.
* A copy of the License is located at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* or in the "license" file accompanying this file. This file is distributed
* on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either
* express or implied. See the License for the specific language governing
* permissions and limitations under the License.
* Copyright OpenSearch Contributors
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.opensearch.response.agg;

import java.util.HashMap;
Expand Down
13 changes: 3 additions & 10 deletions ...h/src/main/java/org/opensearch/sql/opensearch/response/agg/NoBucketAggregationParser.java
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,16 +1,9 @@
/*
* Licensed under the Apache License, Version 2.0 (the "License").
* You may not use this file except in compliance with the License.
* A copy of the License is located at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* or in the "license" file accompanying this file. This file is distributed
* on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either
* express or implied. See the License for the specific language governing
* permissions and limitations under the License.
* Copyright OpenSearch Contributors
* SPDX-License-Identifier: Apache-2.0
*/


package org.opensearch.sql.opensearch.response.agg;

import java.util.Arrays;
Expand Down
Loading
Loading