Mailing-List: contact issues-help@flink.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@flink.apache.org
From: rmetzger <git@git.apache.org>
To: issues@flink.incubator.apache.org
Reply-To: issues@flink.incubator.apache.org
References: <git-pr-603-flink@git.apache.org>
In-Reply-To: <git-pr-603-flink@git.apache.org>
Subject: [GitHub] flink pull request: [FLINK-1753] Extend KafkaITCase,
 reworked Pers...
Content-Type: text/plain
Message-Id: <20150416073324.BBB97E09FB@git1-us-west.apache.org>
Date: Thu, 16 Apr 2015 07:33:24 +0000 (UTC)

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

    https://github.com/apache/flink/pull/603#discussion_r28489103
  
    --- Diff: flink-staging/flink-streaming/flink-streaming-connectors/src/test/java/org/apache/flink/streaming/connectors/kafka/KafkaITCase.java ---
    @@ -709,4 +834,72 @@ private static KafkaServer getKafkaServer(int brokerId, File tmpFolder) throws U
     		private static final long serialVersionUID = 1L;
     	}
     
    +/*	@Test
    +	public void testKafkaWithoutFlink() {
    +
    +		// start consumer:
    +		new Thread(new Runnable() {
    +			@Override
    +			public void run() {
    +				LOG.info("Starting consumer");
    +				// consume from "testtopic"
    +
    +				Properties cprops = new Properties();
    +				cprops.put("zookeeper.connect", zookeeperConnectionString);
    +				cprops.put("group.id", "mygroupid");
    +				cprops.put("zookeeper.session.timeout.ms", "400");
    +				cprops.put("zookeeper.sync.time.ms", "200");
    +				cprops.put("auto.commit.interval.ms", "1000");
    +				ConsumerConnector jcc = Consumer.createJavaConsumerConnector(new ConsumerConfig(cprops));
    +				Map<String,Integer> tcm = new HashMap<String, Integer>(1);
    +				tcm.put("testtopic", 1);
    +				Map<String, List<KafkaStream<byte[], byte[]>>> ms = jcc.createMessageStreams(tcm);
    +				LOG.info("message streams "+ms);
    +				Assert.assertEquals(1, ms.size());
    +
    +				List<KafkaStream<byte[], byte[]>> strList = ms.get("testtopic");
    +				Assert.assertEquals(1, strList.size());
    +
    +				KafkaStream<byte[], byte[]> str = strList.get(0);
    +				LOG.info("str = "+str);
    +
    +				ConsumerIterator<byte[], byte[]> it = str.iterator();
    +				while(it.hasNext()) {
    +					MessageAndMetadata<byte[], byte[]> msg = it.next();
    +					LOG.info("msg = "+msg);
    +				}
    +				LOG.info("Finished consumer");
    +			}
    +		}).start();
    +
    +		// start producer:
    +		Properties props = new Properties();
    +		String brks = "";
    +		for(KafkaServer broker : brokers) {
    +			SocketServer srv = broker.socketServer();
    +			String host = "localhost";
    +			if(srv.host() != null) {
    +				host = srv.host();
    +			}
    +			brks += host+":"+broker.socketServer().port()+",";
    +		}
    +		LOG.info("Using broker list "+brks);
    +		props.setProperty("metadata.broker.list", brks);
    +		props.put("serializer.class", StringEncoder.class.getCanonicalName());
    +
    +		ProducerConfig config = new ProducerConfig(props);
    +		Producer p = new Producer<String, String>(config);
    +
    +		for(int i = 0; i < 10; i++) {
    +			p.send(new KeyedMessage<String, String>("testtopic", "What a message"));
    +		}
    +
    +		LOG.info("+++++++++++ FINISHED PRODUCING ++++++++++++++++ ");
    +
    +		try {
    +			Thread.sleep(1000000);
    +		} catch (InterruptedException e) {
    +			e.printStackTrace();
    +		}
    --- End diff --
    
    I was so desperate that I wanted to test the Kafka stuff without any Flink code. But in the end I figured it out.
    
    I can remove the code.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---