[
https://issues.apache.org/jira/browse/STORM-1736?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15269005#comment-15269005
]
ASF GitHub Bot commented on STORM-1736:
---------------------------------------
Github user abhishekagarwal87 commented on the pull request:
https://github.com/apache/storm/pull/1376#issuecomment-216587066
@harshach - with this change, test compilation will fail once the #1386 is
merged. Since this is the internal class used only in test, do we really need
to make it compatible with 0.10 branch?
> Change KafkaTestBroker.buildKafkaConfig to new KafkaConfig api.
> ---------------------------------------------------------------
>
> Key: STORM-1736
> URL: https://issues.apache.org/jira/browse/STORM-1736
> Project: Apache Storm
> Issue Type: Bug
> Reporter: Sriharsha Chintalapani
> Assignee: Sriharsha Chintalapani
>
> 1.x-branch failing with following error
> [ERROR] Failed to execute goal
> org.apache.maven.plugins:maven-compiler-plugin:3.1:testCompile
> (default-testCompile) on project storm-kafka: Compilation failure
> [ERROR]
> /Users/harsha/code/harshach/incubator-storm/external/storm-kafka/src/test/org/apache/storm/kafka/KafkaTestBroker.java:[70,16]
> constructor KafkaConfig in class kafka.server.KafkaConfig cannot be applied
> to given types;
> [ERROR] required: java.util.Map<?,?>,boolean
> [ERROR] found: java.util.Properties
> [ERROR] reason: actual and formal argument lists differ in length
> [ERROR] -> [Help 1]
> [ERROR]
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
