You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2017/10/10 13:03:00 UTC

[jira] [Commented] (FLINK-7652) Port CurrentJobIdsHandler to new REST endpoint

    [ https://issues.apache.org/jira/browse/FLINK-7652?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16198634#comment-16198634 ] 

ASF GitHub Bot commented on FLINK-7652:
---------------------------------------

Github user tillrohrmann commented on a diff in the pull request:

    https://github.com/apache/flink/pull/4734#discussion_r143718709
  
    --- 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> {
    +
    +		private static final long serialVersionUID = 1009738627775768331L;
    +
    +		public JobStatusesWithIdsOverviewSerializer() {
    +			super(JobStatusesWithIdsOverview.class);
    +		}
    +
    +		@Override
    +		public void serialize(
    +				JobStatusesWithIdsOverview jobStatusesWithIdsOverview,
    +				JsonGenerator jsonGenerator,
    +				SerializerProvider serializerProvider) throws IOException {
    +
    +			jsonGenerator.writeStartObject();
    +
    +			jsonGenerator.writeArrayFieldStart(FIELD_NAME_JOBS_CREATED_IDS);
    +			for (JobID jid : jobStatusesWithIdsOverview.getJobsCreated()) {
    +				jsonGenerator.writeString(jid.toString());
    +			}
    +			jsonGenerator.writeEndArray();
    +
    +			jsonGenerator.writeArrayFieldStart(FIELD_NAME_JOBS_RUNNING_IDS);
    +			for (JobID jid : jobStatusesWithIdsOverview.getJobsRunningOrPending()) {
    +				jsonGenerator.writeString(jid.toString());
    +			}
    +			jsonGenerator.writeEndArray();
    +
    +			jsonGenerator.writeArrayFieldStart(FIELD_NAME_JOBS_FINISHED_IDS);
    +			for (JobID jid : jobStatusesWithIdsOverview.getJobsFinished()) {
    +				jsonGenerator.writeString(jid.toString());
    +			}
    +			jsonGenerator.writeEndArray();
    +
    +			jsonGenerator.writeArrayFieldStart(FIELD_NAME_JOBS_CANCELLING_IDS);
    +			for (JobID jid : jobStatusesWithIdsOverview.getJobsCancelling()) {
    +				jsonGenerator.writeString(jid.toString());
    +			}
    +			jsonGenerator.writeEndArray();
    +
    +			jsonGenerator.writeArrayFieldStart(FIELD_NAME_JOBS_CANCELLED_IDS);
    +			for (JobID jid : jobStatusesWithIdsOverview.getJobsCancelled()) {
    +				jsonGenerator.writeString(jid.toString());
    +			}
    +			jsonGenerator.writeEndArray();
    +
    +			jsonGenerator.writeArrayFieldStart(FIELD_NAME_JOBS_FAILING_IDS);
    +			for (JobID jid : jobStatusesWithIdsOverview.getJobsFailing()) {
    +				jsonGenerator.writeString(jid.toString());
    +			}
    +			jsonGenerator.writeEndArray();
    +
    +			jsonGenerator.writeArrayFieldStart(FIELD_NAME_JOBS_FAILED_IDS);
    +			for (JobID jid : jobStatusesWithIdsOverview.getJobsFailed()) {
    +				jsonGenerator.writeString(jid.toString());
    +			}
    +			jsonGenerator.writeEndArray();
    +
    +			jsonGenerator.writeArrayFieldStart(FIELD_NAME_JOBS_RESTARTING_IDS);
    +			for (JobID jid : jobStatusesWithIdsOverview.getJobsRestarting()) {
    +				jsonGenerator.writeString(jid.toString());
    +			}
    +			jsonGenerator.writeEndArray();
    +
    +			jsonGenerator.writeArrayFieldStart(FIELD_NAME_JOBS_SUSPENDED_IDS);
    +			for (JobID jid : jobStatusesWithIdsOverview.getJobsSuspended()) {
    +				jsonGenerator.writeString(jid.toString());
    +			}
    +			jsonGenerator.writeEndArray();
    +
    +			jsonGenerator.writeArrayFieldStart(FIELD_NAME_JOBS_RECONCILING_IDS);
    +			for (JobID jid : jobStatusesWithIdsOverview.getJobsReconciling()) {
    +				jsonGenerator.writeString(jid.toString());
    +			}
    +			jsonGenerator.writeEndArray();
    +
    +			jsonGenerator.writeEndObject();
    +		}
    +	}
    +
    +	public static final class JobStatusesWithIdsOverviewDeserializer extends StdDeserializer<JobStatusesWithIdsOverview> {
    +
    +		private static final long serialVersionUID = 4733739296433630723L;
    +
    +		public JobStatusesWithIdsOverviewDeserializer() {
    +			super(JobStatusesWithIdsOverview.class);
    +		}
    +
    +		@Override
    +		public JobStatusesWithIdsOverview deserialize(
    +				JsonParser jsonParser,
    +				DeserializationContext deserializationContext) throws IOException {
    +
    +			JsonNode rootNode = jsonParser.readValueAsTree();
    +
    +			List<JobID> jobsCreated = new LinkedList<>();
    --- End diff --
    
    I think we should rather use `ArrayList` instead of `LinkedList`.


> 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)