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

Fix missing space in string literal and spurious Javadoc @param tags from LGTM (#8491) 

* Fix missing space in string literal

* Fix spurious Javadoc @param tags
This commit is contained in:
Benedict Jin 2019-09-16 17:07:47 +08:00 committed by Nishant Bangarwa
parent c254261e73
commit c6f4f09557
3 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
core/src/main/java/org/apache/druid/guice/LifecycleModule.java
View File

@ -57,7 +57,7 @@ public class LifecycleModule implements Module
* it is not clear which is actually the best approach. This is more explicit, but eager bindings inside of modules * it is not clear which is actually the best approach. This is more explicit, but eager bindings inside of modules
* is less error-prone. * is less error-prone.
* *
* @param clazz, the class to instantiate * @param clazz the class to instantiate
* @return this, for chaining. * @return this, for chaining.
*/ */
public static void register(Binder binder, Class<?> clazz) public static void register(Binder binder, Class<?> clazz)
@ -78,7 +78,7 @@ public class LifecycleModule implements Module
* it is not clear which is actually the best approach. This is more explicit, but eager bindings inside of modules * it is not clear which is actually the best approach. This is more explicit, but eager bindings inside of modules
* is less error-prone. * is less error-prone.
* *
* @param clazz, the class to instantiate * @param clazz the class to instantiate
* @param annotation The annotation class to register with Guice * @param annotation The annotation class to register with Guice
* @return this, for chaining * @return this, for chaining
*/ */

2
extensions-core/kinesis-indexing-service/src/main/java/org/apache/druid/indexing/kinesis/KinesisRecordSupplier.java
View File

@ -319,7 +319,7 @@ public class KinesisRecordSupplier implements RecordSupplier<String, String>
catch (RejectedExecutionException e) { catch (RejectedExecutionException e) {
log.warn( log.warn(
e, e,
"Caught RejectedExecutionException, KinesisRecordSupplier for partition[%s] has likely temporarily shutdown the ExecutorService." "Caught RejectedExecutionException, KinesisRecordSupplier for partition[%s] has likely temporarily shutdown the ExecutorService. "
+ "This is expected behavior after calling seek(), seekToEarliest() and seekToLatest()", + "This is expected behavior after calling seek(), seekToEarliest() and seekToLatest()",
streamPartition.getPartitionId() streamPartition.getPartitionId()
); );

2
processing/src/main/java/org/apache/druid/query/scan/ScanQueryRunnerFactory.java
View File

@ -295,7 +295,7 @@ public class ScanQueryRunnerFactory implements QueryRunnerFactory<ScanResultValu
descriptorsOrdered = Collections.singletonList(((SpecificSegmentSpec) spec).getDescriptor().getInterval()); descriptorsOrdered = Collections.singletonList(((SpecificSegmentSpec) spec).getDescriptor().getInterval());
} else { } else {
throw new UOE( throw new UOE(
"Time-ordering on scan queries is only supported for queries with segment specs" "Time-ordering on scan queries is only supported for queries with segment specs "
+ "of type MultipleSpecificSegmentSpec or SpecificSegmentSpec...a [%s] was received instead.", + "of type MultipleSpecificSegmentSpec or SpecificSegmentSpec...a [%s] was received instead.",
spec.getClass().getSimpleName() spec.getClass().getSimpleName()
); );