[
https://issues.apache.org/jira/browse/BEAM-11411?focusedWorklogId=541856&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-541856
]
ASF GitHub Bot logged work on BEAM-11411:
-----------------------------------------
Author: ASF GitHub Bot
Created on: 26/Jan/21 04:26
Start Date: 26/Jan/21 04:26
Worklog Time Spent: 10m
Work Description: ramazan-yapparov commented on a change in pull request
#13636:
URL: https://github.com/apache/beam/pull/13636#discussion_r563605230
##########
File path:
examples/java/src/main/java/org/apache/beam/examples/complete/kafkatopubsub/kafka/consumer/Utils.java
##########
@@ -162,4 +165,11 @@ public static boolean isSslSpecified(KafkaToPubsubOptions
options) {
|| options.getKeystorePath() != null
|| options.getKeyPassword() != null;
}
+
+ public static Map<String, Object> parseKafkaConsumerConfig(String
kafkaConsumerConfig) {
+ return Arrays.stream(kafkaConsumerConfig.split(";"))
+ .map(s -> s.split("="))
+ .map(kv -> Pair.of(kv[0], kv[1]))
+ .collect(Collectors.toMap(Pair::getKey, Pair::getValue));
Review comment:
Removed `Pair` related code, but left method in place.
This class is created for such methods, in my opinion it is better to
declare them in `Utils` in order to leave main class as clean as possible
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 541856)
Time Spent: 14h 40m (was: 14.5h)
> Example of E2E test for Pub/Sub environment
> -------------------------------------------
>
> Key: BEAM-11411
> URL: https://issues.apache.org/jira/browse/BEAM-11411
> Project: Beam
> Issue Type: Improvement
> Components: examples-java
> Reporter: Ilya Kozyrev
> Priority: P2
> Fix For: 2.28.0
>
> Time Spent: 14h 40m
> Remaining Estimate: 0h
>
> Kafka to Pub/Sub pipeline [example|https://github.com/apache/beam/pull/13112]
> needs to have e2e tests. However, in Beam Testing Guide
> [https://cwiki.apache.org/confluence/display/BEAM/Contribution+Testing+Guide]
> , we have not found examples or documentation for e2e tests that require a
> Pub/Sub environment can be set up.
> During the investigation and with the help of our reviewers we found possible
> approaches for our issue.
> For Pub/Sub, e2e test could use [Pub/Sub
> emulator|https://cloud.google.com/pubsub/docs/emulator]
> These e2e test examples can be referenced in the
> [https://cwiki.apache.org/confluence/display/BEAM/Contribution+Testing+Guide]
> to serve examples of how customers may approach e2e testing of pipelines in
> their environments.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
