flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From zentol <...@git.apache.org>
Subject [GitHub] flink pull request #4802: [FLINK-7791] [REST][client] Integrate LIST command...
Date Wed, 11 Oct 2017 13:25:59 GMT
Github user zentol commented on a diff in the pull request:

    https://github.com/apache/flink/pull/4802#discussion_r144009864
  
    --- Diff: flink-clients/src/test/java/org/apache/flink/client/program/ClusterClientTest.java
---
    @@ -134,6 +140,25 @@ public void testClusterClientSavepoint() throws Exception {
     		}
     	}
     
    +	@Test
    +	public void testClusterClientList() throws Exception {
    +		Configuration config = new Configuration();
    +		config.setString(JobManagerOptions.ADDRESS, "localhost");
    +
    +		TestListActorGateway gateway = new TestListActorGateway();
    +		ClusterClient clusterClient = new TestClusterClient(config, gateway);
    +		try {
    +			CompletableFuture<Collection<JobDetails>> jobDetailsFuture = clusterClient.listJobs();
    +			Collection<JobDetails> jobDetails = jobDetailsFuture.get();
    +			Assert.assertTrue(gateway.messageArrived);
    +			// finished jobs should be ignored
    +			Assert.assertEquals(1, jobDetails.size());
    --- End diff --
    
    No, the LIST command only lists running/scheduled jobs.


---

Mime
View raw message