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

Remove `@JsonIgnore` annotations for private members of `TaskAction` classes (#16099) 

* Remove @JsonIgnore annotations for private members

* checkstyle fix - removed unused imports.
This commit is contained in:
Abhishek Radhakrishnan 2024-03-12 02:42:36 +08:00 committed by GitHub
parent 94d2a28465
commit 8084f2206b
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
10 changed files with 0 additions and 38 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
indexing-service/src/main/java/org/apache/druid/indexing/common/actions/LockReleaseAction.java
View File

@ -20,7 +20,6 @@
package org.apache.druid.indexing.common.actions; package org.apache.druid.indexing.common.actions;
import com.fasterxml.jackson.annotation.JsonCreator; import com.fasterxml.jackson.annotation.JsonCreator;
import com.fasterxml.jackson.annotation.JsonIgnore;
import com.fasterxml.jackson.annotation.JsonProperty; import com.fasterxml.jackson.annotation.JsonProperty;
import com.fasterxml.jackson.core.type.TypeReference; import com.fasterxml.jackson.core.type.TypeReference;
import org.apache.druid.indexing.common.task.Task; import org.apache.druid.indexing.common.task.Task;
@ -28,7 +27,6 @@ import org.joda.time.Interval;
public class LockReleaseAction implements TaskAction<Void> public class LockReleaseAction implements TaskAction<Void>
{ {
@JsonIgnore
private final Interval interval; private final Interval interval;
@JsonCreator @JsonCreator

4
indexing-service/src/main/java/org/apache/druid/indexing/common/actions/MarkSegmentsAsUnusedAction.java
View File

@ -20,7 +20,6 @@
package org.apache.druid.indexing.common.actions; package org.apache.druid.indexing.common.actions;
import com.fasterxml.jackson.annotation.JsonCreator; import com.fasterxml.jackson.annotation.JsonCreator;
import com.fasterxml.jackson.annotation.JsonIgnore;
import com.fasterxml.jackson.annotation.JsonProperty; import com.fasterxml.jackson.annotation.JsonProperty;
import com.fasterxml.jackson.core.type.TypeReference; import com.fasterxml.jackson.core.type.TypeReference;
import org.apache.druid.indexing.common.task.Task; import org.apache.druid.indexing.common.task.Task;
@ -28,10 +27,7 @@ import org.joda.time.Interval;
public class MarkSegmentsAsUnusedAction implements TaskAction<Integer> public class MarkSegmentsAsUnusedAction implements TaskAction<Integer>
{ {
@JsonIgnore
private final String dataSource; private final String dataSource;
@JsonIgnore
private final Interval interval; private final Interval interval;
@JsonCreator @JsonCreator

8
indexing-service/src/main/java/org/apache/druid/indexing/common/actions/RetrieveUnusedSegmentsAction.java
View File

@ -20,7 +20,6 @@
package org.apache.druid.indexing.common.actions; package org.apache.druid.indexing.common.actions;
import com.fasterxml.jackson.annotation.JsonCreator; import com.fasterxml.jackson.annotation.JsonCreator;
import com.fasterxml.jackson.annotation.JsonIgnore;
import com.fasterxml.jackson.annotation.JsonProperty; import com.fasterxml.jackson.annotation.JsonProperty;
import com.fasterxml.jackson.core.type.TypeReference; import com.fasterxml.jackson.core.type.TypeReference;
import org.apache.druid.indexing.common.task.Task; import org.apache.druid.indexing.common.task.Task;
@ -34,16 +33,9 @@ import java.util.List;
public class RetrieveUnusedSegmentsAction implements TaskAction<List<DataSegment>> public class RetrieveUnusedSegmentsAction implements TaskAction<List<DataSegment>>
{ {
@JsonIgnore
private final String dataSource; private final String dataSource;
@JsonIgnore
private final Interval interval; private final Interval interval;
@JsonIgnore
private final Integer limit; private final Integer limit;
@JsonIgnore
private final DateTime maxUsedStatusLastUpdatedTime; private final DateTime maxUsedStatusLastUpdatedTime;
@JsonCreator @JsonCreator

6
indexing-service/src/main/java/org/apache/druid/indexing/common/actions/RetrieveUsedSegmentsAction.java
View File

@ -20,7 +20,6 @@
package org.apache.druid.indexing.common.actions; package org.apache.druid.indexing.common.actions;
import com.fasterxml.jackson.annotation.JsonCreator; import com.fasterxml.jackson.annotation.JsonCreator;
import com.fasterxml.jackson.annotation.JsonIgnore;
import com.fasterxml.jackson.annotation.JsonProperty; import com.fasterxml.jackson.annotation.JsonProperty;
import com.fasterxml.jackson.core.type.TypeReference; import com.fasterxml.jackson.core.type.TypeReference;
import com.google.common.base.Preconditions; import com.google.common.base.Preconditions;
@ -67,13 +66,8 @@ public class RetrieveUsedSegmentsAction implements TaskAction<Collection<DataSeg
{ {
private static final Logger log = new Logger(RetrieveUsedSegmentsAction.class); private static final Logger log = new Logger(RetrieveUsedSegmentsAction.class);
@JsonIgnore
private final String dataSource; private final String dataSource;
@JsonIgnore
private final List<Interval> intervals; private final List<Interval> intervals;
@JsonIgnore
private final Segments visibility; private final Segments visibility;
@JsonCreator @JsonCreator

2
indexing-service/src/main/java/org/apache/druid/indexing/common/actions/SegmentMetadataUpdateAction.java
View File

@ -20,7 +20,6 @@
package org.apache.druid.indexing.common.actions; package org.apache.druid.indexing.common.actions;
import com.fasterxml.jackson.annotation.JsonCreator; import com.fasterxml.jackson.annotation.JsonCreator;
import com.fasterxml.jackson.annotation.JsonIgnore;
import com.fasterxml.jackson.annotation.JsonProperty; import com.fasterxml.jackson.annotation.JsonProperty;
import com.fasterxml.jackson.core.type.TypeReference; import com.fasterxml.jackson.core.type.TypeReference;
import com.google.common.collect.ImmutableSet; import com.google.common.collect.ImmutableSet;
@ -38,7 +37,6 @@ import java.util.stream.Collectors;
public class SegmentMetadataUpdateAction implements TaskAction<Void> public class SegmentMetadataUpdateAction implements TaskAction<Void>
{ {
@JsonIgnore
private final Set<DataSegment> segments; private final Set<DataSegment> segments;
@JsonCreator @JsonCreator

2
indexing-service/src/main/java/org/apache/druid/indexing/common/actions/SegmentNukeAction.java
View File

@ -20,7 +20,6 @@
package org.apache.druid.indexing.common.actions; package org.apache.druid.indexing.common.actions;
import com.fasterxml.jackson.annotation.JsonCreator; import com.fasterxml.jackson.annotation.JsonCreator;
import com.fasterxml.jackson.annotation.JsonIgnore;
import com.fasterxml.jackson.annotation.JsonProperty; import com.fasterxml.jackson.annotation.JsonProperty;
import com.fasterxml.jackson.core.type.TypeReference; import com.fasterxml.jackson.core.type.TypeReference;
import com.google.common.collect.ImmutableSet; import com.google.common.collect.ImmutableSet;
@ -38,7 +37,6 @@ import java.util.stream.Collectors;
public class SegmentNukeAction implements TaskAction<Void> public class SegmentNukeAction implements TaskAction<Void>
{ {
@JsonIgnore
private final Set<DataSegment> segments; private final Set<DataSegment> segments;
@JsonCreator @JsonCreator

5
indexing-service/src/main/java/org/apache/druid/indexing/common/actions/TimeChunkLockAcquireAction.java
View File

@ -20,7 +20,6 @@
package org.apache.druid.indexing.common.actions; package org.apache.druid.indexing.common.actions;
import com.fasterxml.jackson.annotation.JsonCreator; import com.fasterxml.jackson.annotation.JsonCreator;
import com.fasterxml.jackson.annotation.JsonIgnore;
import com.fasterxml.jackson.annotation.JsonProperty; import com.fasterxml.jackson.annotation.JsonProperty;
import com.fasterxml.jackson.core.type.TypeReference; import com.fasterxml.jackson.core.type.TypeReference;
import com.google.common.base.Preconditions; import com.google.common.base.Preconditions;
@ -41,11 +40,7 @@ import javax.annotation.Nullable;
public class TimeChunkLockAcquireAction implements TaskAction<TaskLock> public class TimeChunkLockAcquireAction implements TaskAction<TaskLock>
{ {
private final TaskLockType type; private final TaskLockType type;
@JsonIgnore
private final Interval interval; private final Interval interval;
@JsonIgnore
private final long timeoutMs; private final long timeoutMs;
@JsonCreator @JsonCreator

4
indexing-service/src/main/java/org/apache/druid/indexing/common/actions/TimeChunkLockTryAcquireAction.java
View File

@ -20,7 +20,6 @@
package org.apache.druid.indexing.common.actions; package org.apache.druid.indexing.common.actions;
import com.fasterxml.jackson.annotation.JsonCreator; import com.fasterxml.jackson.annotation.JsonCreator;
import com.fasterxml.jackson.annotation.JsonIgnore;
import com.fasterxml.jackson.annotation.JsonProperty; import com.fasterxml.jackson.annotation.JsonProperty;
import com.fasterxml.jackson.core.type.TypeReference; import com.fasterxml.jackson.core.type.TypeReference;
import org.apache.druid.indexing.common.TaskLock; import org.apache.druid.indexing.common.TaskLock;
@ -38,10 +37,7 @@ import javax.annotation.Nullable;
*/ */
public class TimeChunkLockTryAcquireAction implements TaskAction<TaskLock> public class TimeChunkLockTryAcquireAction implements TaskAction<TaskLock>
{ {
@JsonIgnore
private final TaskLockType type; private final TaskLockType type;
@JsonIgnore
private final Interval interval; private final Interval interval;
@JsonCreator @JsonCreator

2
indexing-service/src/main/java/org/apache/druid/indexing/common/actions/UpdateLocationAction.java
View File

@ -20,7 +20,6 @@
package org.apache.druid.indexing.common.actions; package org.apache.druid.indexing.common.actions;
import com.fasterxml.jackson.annotation.JsonCreator; import com.fasterxml.jackson.annotation.JsonCreator;
import com.fasterxml.jackson.annotation.JsonIgnore;
import com.fasterxml.jackson.annotation.JsonProperty; import com.fasterxml.jackson.annotation.JsonProperty;
import com.fasterxml.jackson.core.type.TypeReference; import com.fasterxml.jackson.core.type.TypeReference;
import com.google.common.base.Optional; import com.google.common.base.Optional;
@ -30,7 +29,6 @@ import org.apache.druid.indexing.overlord.TaskRunner;
public class UpdateLocationAction implements TaskAction<Void> public class UpdateLocationAction implements TaskAction<Void>
{ {
@JsonIgnore
private final TaskLocation taskLocation; private final TaskLocation taskLocation;
@JsonCreator @JsonCreator

3
indexing-service/src/main/java/org/apache/druid/indexing/common/actions/UpdateStatusAction.java
View File

@ -20,7 +20,6 @@
package org.apache.druid.indexing.common.actions; package org.apache.druid.indexing.common.actions;
import com.fasterxml.jackson.annotation.JsonCreator; import com.fasterxml.jackson.annotation.JsonCreator;
import com.fasterxml.jackson.annotation.JsonIgnore;
import com.fasterxml.jackson.annotation.JsonProperty; import com.fasterxml.jackson.annotation.JsonProperty;
import com.fasterxml.jackson.core.type.TypeReference; import com.fasterxml.jackson.core.type.TypeReference;
import com.google.common.base.Optional; import com.google.common.base.Optional;
@ -32,9 +31,7 @@ import java.util.Objects;
public class UpdateStatusAction implements TaskAction<Void> public class UpdateStatusAction implements TaskAction<Void>
{ {
@JsonIgnore
private final String status; private final String status;
@JsonIgnore
private final TaskStatus statusFull; private final TaskStatus statusFull;
@Deprecated @Deprecated