[ https://issues.apache.org/jira/browse/FLINK-7652?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16200120#comment-16200120 ]
ASF GitHub Bot commented on FLINK-7652: --------------------------------------- Github user tzulitai commented on a diff in the pull request: https://github.com/apache/flink/pull/4734#discussion_r143983019 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/messages/webmonitor/JobStatusesWithIdsOverview.java --- @@ -0,0 +1,368 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreements. See the NOTICE file + * distributed with this work for additional information + * regarding copyright ownership. The ASF licenses this file + * to you under the Apache License, Version 2.0 (the + * "License"); you may not use this file except in compliance + * with the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License 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. + */ + +package org.apache.flink.runtime.messages.webmonitor; + +import org.apache.flink.api.common.JobID; +import org.apache.flink.runtime.rest.messages.ResponseBody; + +import com.fasterxml.jackson.core.JsonGenerator; +import com.fasterxml.jackson.core.JsonParser; +import com.fasterxml.jackson.databind.DeserializationContext; +import com.fasterxml.jackson.databind.JsonNode; +import com.fasterxml.jackson.databind.SerializerProvider; +import com.fasterxml.jackson.databind.annotation.JsonDeserialize; +import com.fasterxml.jackson.databind.annotation.JsonSerialize; +import com.fasterxml.jackson.databind.deser.std.StdDeserializer; +import com.fasterxml.jackson.databind.ser.std.StdSerializer; + +import java.io.IOException; +import java.util.ArrayList; +import java.util.LinkedList; +import java.util.List; + +import static org.apache.flink.util.Preconditions.checkNotNull; + +/** + * An overview of how many jobs are in which status. + */ +@JsonSerialize(using = JobStatusesWithIdsOverview.JobStatusesWithIdsOverviewSerializer.class) +@JsonDeserialize(using = JobStatusesWithIdsOverview.JobStatusesWithIdsOverviewDeserializer.class) +public class JobStatusesWithIdsOverview implements ResponseBody, InfoMessage { + + private static final long serialVersionUID = -3699051943490133183L; + + public static final String FIELD_NAME_JOBS_CREATED_IDS = "jobs-created"; + public static final String FIELD_NAME_JOBS_RUNNING_IDS = "jobs-running"; + public static final String FIELD_NAME_JOBS_FINISHED_IDS = "jobs-finished"; + public static final String FIELD_NAME_JOBS_CANCELLING_IDS = "jobs-cancelling"; + public static final String FIELD_NAME_JOBS_CANCELLED_IDS = "jobs-cancelled"; + public static final String FIELD_NAME_JOBS_FAILING_IDS = "jobs-failing"; + public static final String FIELD_NAME_JOBS_FAILED_IDS = "jobs-failed"; + public static final String FIELD_NAME_JOBS_RESTARTING_IDS = "jobs-restarting"; + public static final String FIELD_NAME_JOBS_SUSPENDED_IDS = "jobs-suspended"; + public static final String FIELD_NAME_JOBS_RECONCILING_IDS = "jobs-reconciling"; + + private final List<JobID> jobsCreated; + private final List<JobID> jobsRunningOrPending; + private final List<JobID> jobsFinished; + private final List<JobID> jobsCancelling; + private final List<JobID> jobsCancelled; + private final List<JobID> jobsFailing; + private final List<JobID> jobsFailed; + private final List<JobID> jobsRestarting; + private final List<JobID> jobsSuspended; + private final List<JobID> jobsReconciling; + + public JobStatusesWithIdsOverview( + List<JobID> jobsCreated, + List<JobID> jobsRunningOrPending, + List<JobID> jobsFinished, + List<JobID> jobsCancelling, + List<JobID> jobsCancelled, + List<JobID> jobsFailing, + List<JobID> jobsFailed, + List<JobID> jobsRestarting, + List<JobID> jobsSuspended, + List<JobID> jobsReconciling) { + + this.jobsCreated = checkNotNull(jobsCreated); + this.jobsRunningOrPending = checkNotNull(jobsRunningOrPending); + this.jobsFinished = checkNotNull(jobsFinished); + this.jobsCancelling = checkNotNull(jobsCancelling); + this.jobsCancelled = checkNotNull(jobsCancelled); + this.jobsFailing = checkNotNull(jobsFailing); + this.jobsFailed = checkNotNull(jobsFailed); + this.jobsRestarting = checkNotNull(jobsRestarting); + this.jobsSuspended = checkNotNull(jobsSuspended); + this.jobsReconciling = checkNotNull(jobsReconciling); + } + + public JobStatusesWithIdsOverview(JobStatusesWithIdsOverview first, JobStatusesWithIdsOverview second) { + this.jobsCreated = combine(first.getJobsCreated(), second.getJobsCreated()); + this.jobsRunningOrPending = combine(first.getJobsRunningOrPending(), second.getJobsRunningOrPending()); + this.jobsFinished = combine(first.getJobsFinished(), second.getJobsFinished()); + this.jobsCancelling = combine(first.getJobsCancelling(), second.getJobsCancelling()); + this.jobsCancelled = combine(first.getJobsCancelled(), second.getJobsCancelled()); + this.jobsFailing = combine(first.getJobsFailing(), second.getJobsFailing()); + this.jobsFailed = combine(first.getJobsFailed(), second.getJobsFailed()); + this.jobsRestarting = combine(first.getJobsRestarting(), second.getJobsRestarting()); + this.jobsSuspended = combine(first.getJobsSuspended(), second.getJobsSuspended()); + this.jobsReconciling = combine(first.getJobsReconciling(), second.getJobsReconciling()); + } + + public List<JobID> getJobsCreated() { + return jobsCreated; + } + + public List<JobID> getJobsRunningOrPending() { + return jobsRunningOrPending; + } + + public List<JobID> getJobsFinished() { + return jobsFinished; + } + + public List<JobID> getJobsCancelling() { + return jobsCancelling; + } + + public List<JobID> getJobsCancelled() { + return jobsCancelled; + } + + public List<JobID> getJobsFailing() { + return jobsFailing; + } + + public List<JobID> getJobsFailed() { + return jobsFailed; + } + + public List<JobID> getJobsRestarting() { + return jobsRestarting; + } + + public List<JobID> getJobsSuspended() { + return jobsSuspended; + } + + public List<JobID> getJobsReconciling() { + return jobsReconciling; + } + + // ------------------------------------------------------------------------ + + + @Override + public int hashCode() { + return jobsCreated.hashCode() ^ + jobsRunningOrPending.hashCode() ^ + jobsFinished.hashCode() ^ + jobsCancelling.hashCode() ^ + jobsCancelled.hashCode() ^ + jobsFailing.hashCode() ^ + jobsFailed.hashCode() ^ + jobsRestarting.hashCode() ^ + jobsSuspended.hashCode() ^ + jobsReconciling.hashCode(); + } + + @Override + public boolean equals(Object obj) { + if (obj == this) { + return true; + } + else if (obj instanceof JobStatusesWithIdsOverview) { + JobStatusesWithIdsOverview that = (JobStatusesWithIdsOverview) obj; + return this.jobsCreated.equals(that.jobsCreated) && + this.jobsRunningOrPending.equals(that.jobsRunningOrPending) && + this.jobsFinished.equals(that.jobsFinished) && + this.jobsCancelling.equals(that.jobsCancelling) && + this.jobsCancelled.equals(that.jobsCancelled) && + this.jobsFailing.equals(that.jobsFailing) && + this.jobsFailed.equals(that.jobsFailed) && + this.jobsRestarting.equals(that.jobsRestarting) && + this.jobsSuspended.equals(that.jobsSuspended) && + this.jobsReconciling.equals(that.jobsReconciling); + } + else { + return false; + } + } + + @Override + public String toString() { + return "JobStatusesWithIdsOverview {" + + "createdJobs=" + jobsCreated + + ", runningOrPendingJobs=" + jobsRunningOrPending + + ", finishedJobs=" + jobsFinished + + ", cancellingJobs=" + jobsCancelling + + ", cancelledJobs=" + jobsCancelled + + ", failingJobs=" + jobsFailing + + ", failedJobs=" + jobsFailed + + ", restartingJobs=" + jobsRestarting + + ", suspendedJobs=" + jobsSuspended + + ", reconcilingJobs=" + jobsReconciling + + '}'; + } + + // ------------------------------------------------------------------------ + // Message serializers + // ------------------------------------------------------------------------ + + public static final class JobStatusesWithIdsOverviewSerializer extends StdSerializer<JobStatusesWithIdsOverview> { --- End diff -- I think it was due to the serialization of `JobID`s. Maybe what should be enough is to define a (de)serializer for the `JobID` class? > Port CurrentJobIdsHandler to new REST endpoint > ---------------------------------------------- > > Key: FLINK-7652 > URL: https://issues.apache.org/jira/browse/FLINK-7652 > Project: Flink > Issue Type: Sub-task > Components: REST, Webfrontend > Reporter: Tzu-Li (Gordon) Tai > Assignee: Tzu-Li (Gordon) Tai > Labels: flip-6 > Fix For: 1.4.0 > > > Port existing {{CurrentJobIdsHandler}} to new REST endpoint -- This message was sent by Atlassian JIRA (v6.4.14#64029)