Apache
/
druid
mirror of https://github.com/apache/druid.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix non org.apache.druid files and add package name checkstyle rule (#6367) 

* Fix non org.apache.druid files and add package name checkstyle rule

* PR comment
This commit is contained in:
Jonathan Wei 2018-09-21 17:58:19 -07:00 committed by Slim Bouguerra
parent 399a5659b2
commit 364bf9d1f9
8 changed files with 25 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
codestyle/checkstyle.xml
View File

@ -193,5 +193,10 @@
<property name="illegalPattern" value="true"/>
<property name="message" value="Use org.apache.druid.common.config.NullHandling.nullToEmptyIfNeeded instead"/>
</module>
<module name="PackageName">
<property name="format" value="^org.apache.druid.*$"/>
</module>
<module name="PackageDeclaration"/>
</module>
</module>

5
extensions-contrib/parquet-extensions/src/main/java/org/apache/parquet/avro/DruidParquetReadSupport.java
View File

@ -17,7 +17,9 @@
* under the License.
*/
//CHECKSTYLE.OFF: PackageName
package org.apache.parquet.avro;
//CHECKSTYLE.ON: PackageName
import com.google.common.collect.Lists;
import com.google.common.collect.Sets;
@ -37,6 +39,9 @@ import java.util.List;
import java.util.Map;
import java.util.Set;
/**
* This class must in package org.apache.parquet.avro to access the AvroRecordMaterializer constructor
*/
public class DruidParquetReadSupport extends AvroReadSupport<GenericRecord>
{

4
extensions-contrib/statsd-emitter/src/test/java/DimensionConverterTest.java → extensions-contrib/statsd-emitter/src/test/java/org/apache/druid/emitter/statsd/DimensionConverterTest.java
View File

@ -17,10 +17,10 @@
* under the License.
*/
package org.apache.druid.emitter.statsd;
import com.fasterxml.jackson.databind.ObjectMapper;
import com.google.common.collect.ImmutableList;
import org.apache.druid.emitter.statsd.DimensionConverter;
import org.apache.druid.emitter.statsd.StatsDMetric;
import org.apache.druid.java.util.common.DateTimes;
import org.apache.druid.java.util.emitter.service.ServiceMetricEvent;
import org.junit.Test;

4
extensions-contrib/statsd-emitter/src/test/java/StatsDEmitterTest.java → extensions-contrib/statsd-emitter/src/test/java/org/apache/druid/emitter/statsd/StatsDEmitterTest.java
View File

@ -17,11 +17,11 @@
* under the License.
*/
package org.apache.druid.emitter.statsd;
import com.fasterxml.jackson.databind.ObjectMapper;
import org.apache.druid.java.util.emitter.service.ServiceMetricEvent;
import com.timgroup.statsd.StatsDClient;
import org.apache.druid.emitter.statsd.StatsDEmitter;
import org.apache.druid.emitter.statsd.StatsDEmitterConfig;
import org.apache.druid.java.util.common.DateTimes;
import org.junit.Test;

2
extensions-core/hdfs-storage/src/main/java/org/apache/hadoop/fs/HadoopFsWrapper.java
View File

@ -17,7 +17,9 @@
* under the License.
*/
//CHECKSTYLE.OFF: PackageName
package org.apache.hadoop.fs;
//CHECKSTYLE.ON: PackageName
import com.google.common.base.Throwables;
import org.apache.druid.java.util.common.logger.Logger;

6
integration-tests/src/main/java/org/testng/DruidTestRunnerFactory.java
View File

@ -17,7 +17,10 @@
* under the License.
*/
//CHECKSTYLE.OFF: PackageName
package org.testng;
//CHECKSTYLE.ON: PackageName
import com.google.common.base.Throwables;
import com.google.inject.Injector;
@ -43,6 +46,9 @@ import java.net.URL;
import java.nio.charset.StandardCharsets;
import java.util.List;
/**
* This class must be in package org.testng to access protected methods like TestNG.getDefault().getConfiguration()
*/
public class DruidTestRunnerFactory implements ITestRunnerFactory
{
private static final Logger LOG = new Logger(DruidTestRunnerFactory.class);

2
integration-tests/src/test/java/io/druid/tests/security/ITTLSTest.java → integration-tests/src/test/java/org/apache/druid/tests/security/ITTLSTest.java
View File

@ -17,7 +17,7 @@
* under the License.
*/
package io.druid.tests.security;
package org.apache.druid.tests.security;
import com.fasterxml.jackson.databind.ObjectMapper;
import com.google.common.base.Throwables;

5
server/src/test/java/io/druid/segment/realtime/firehose/EventReceiverFirehostIdleTest.java → server/src/test/java/org/apache/druid/segment/realtime/firehose/EventReceiverFirehoseIdleTest.java
View File

@ -17,7 +17,7 @@
* under the License.
*/
package io.druid.segment.realtime.firehose;
package org.apache.druid.segment.realtime.firehose;
import com.google.common.collect.ImmutableList;
import org.apache.commons.io.IOUtils;
@ -26,7 +26,6 @@ import org.apache.druid.data.input.impl.JSONParseSpec;
import org.apache.druid.data.input.impl.MapInputRowParser;
import org.apache.druid.data.input.impl.TimestampSpec;
import org.apache.druid.jackson.DefaultObjectMapper;
import org.apache.druid.segment.realtime.firehose.EventReceiverFirehoseFactory;
import org.apache.druid.server.metrics.EventReceiverFirehoseRegister;
import org.apache.druid.server.security.AllowAllAuthenticator;
import org.apache.druid.server.security.AuthConfig;
@ -39,7 +38,7 @@ import org.junit.Test;
import javax.servlet.http.HttpServletRequest;
import java.util.Locale;
public class EventReceiverFirehostIdleTest
public class EventReceiverFirehoseIdleTest
{
private static final int CAPACITY = 300;
private static final long MAX_IDLE_TIME = 5_000L;