{"action":"created","issue":{"url":"https://api.github.com/repos/TomMD/storm-dynamic-spout/issues/8","repository_url":"https://api.github.com/repos/TomMD/storm-dynamic-spout","labels_url":"https://api.github.com/repos/TomMD/storm-dynamic-spout/issues/8/labels{/name}","comments_url":"https://api.github.com/repos/TomMD/storm-dynamic-spout/issues/8/comments","events_url":"https://api.github.com/repos/TomMD/storm-dynamic-spout/issues/8/events","html_url":"https://github.com/TomMD/storm-dynamic-spout/pull/8","id":324579333,"number":8,"title":"Mirror of salesforce storm-dynamic-spout#82","user":{"login":"TomMD","id":137806,"avatar_url":"https://avatars2.githubusercontent.com/u/137806?v=4","gravatar_id":"","url":"https://api.github.com/users/TomMD","html_url":"https://github.com/TomMD","followers_url":"https://api.github.com/users/TomMD/followers","following_url":"https://api.github.com/users/TomMD/following{/other_user}","gists_url":"https://api.github.com/users/TomMD/gists{/gist_id}","starred_url":"https://api.github.com/users/TomMD/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/TomMD/subscriptions","organizations_url":"https://api.github.com/users/TomMD/orgs","repos_url":"https://api.github.com/users/TomMD/repos","events_url":"https://api.github.com/users/TomMD/events{/privacy}","received_events_url":"https://api.github.com/users/TomMD/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-05-18T23:30:58Z","updated_at":"2018-05-18T23:31:16Z","closed_at":null,"author_association":"OWNER","pull_request":{"url":"https://api.github.com/repos/TomMD/storm-dynamic-spout/pulls/8","html_url":"https://github.com/TomMD/storm-dynamic-spout/pull/8","diff_url":"https://github.com/TomMD/storm-dynamic-spout/pull/8.diff","patch_url":"https://github.com/TomMD/storm-dynamic-spout/pull/8.patch"},"body":"Mirror of salesforce storm-dynamic-spout#82\n**Requirements**\r\n- Needs to allow defining sane defaults.\r\n- Needs to support generating Documentation.\r\n- Needs to allow user to define adhoc settings for any integrations.\r\n\r\n**Questions**\r\n- Do we need to provide a way for integrations to hook in their own configuration definitions?\r\n-- Are these applied ontop of DynamicSpout's config?  What if you have multiple integrations?\r\n\r\nCurrently this is heavily based off of Kafka's configuration (see here: [AbstractConfig](https://github.com/apache/kafka/blob/trunk/clients/src/main/java/org/apache/kafka/common/config/AbstractConfig.java) and [ConsumerConfig](https://github.com/apache/kafka/blob/trunk/clients/src/main/java/org/apache/kafka/clients/consumer/ConsumerConfig.java) )\r\n\r\n\"SpoutConfig\" is a generic Configuration holder.  \r\n\"DynamicSpoutConfig\" extends this supplying a ConfigDefinition, but has no logic beyond defining the ConfigDefinition and a pass thru constructor.\r\n\"SidelineConfig\" makes use of DynamicSpout's ConfigDefinition and appends its own settings to it, again no logic and pass thru constructors.\r\n\r\nNot entirely sure how I feel about this model.  Once we break up Kafka Consumer, how does that fit into this?  Does it need to?\r\n\r\nEven if we go a totally different direction from here, I think this branch gives us a good starting point with concrete classes to play around with.  It was **a lot** more work than I was expecting to convert those maps into concrete classes :/\r\n\r\n\n"},"comment":{"url":"https://api.github.com/repos/TomMD/storm-dynamic-spout/issues/comments/390357797","html_url":"https://github.com/TomMD/storm-dynamic-spout/pull/8#issuecomment-390357797","issue_url":"https://api.github.com/repos/TomMD/storm-dynamic-spout/issues/8","id":390357797,"user":{"login":"inferences[bot]","id":37194012,"avatar_url":"https://avatars2.githubusercontent.com/in/9843?v=4","gravatar_id":"","url":"https://api.github.com/users/inferences%5Bbot%5D","html_url":"https://github.com/apps/inferences","followers_url":"https://api.github.com/users/inferences%5Bbot%5D/followers","following_url":"https://api.github.com/users/inferences%5Bbot%5D/following{/other_user}","gists_url":"https://api.github.com/users/inferences%5Bbot%5D/gists{/gist_id}","starred_url":"https://api.github.com/users/inferences%5Bbot%5D/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/inferences%5Bbot%5D/subscriptions","organizations_url":"https://api.github.com/users/inferences%5Bbot%5D/orgs","repos_url":"https://api.github.com/users/inferences%5Bbot%5D/repos","events_url":"https://api.github.com/users/inferences%5Bbot%5D/events{/privacy}","received_events_url":"https://api.github.com/users/inferences%5Bbot%5D/received_events","type":"Bot","site_admin":false},"created_at":"2018-05-18T23:31:16Z","updated_at":"2018-05-18T23:31:16Z","author_association":"NONE","body":"# Infer analysis failed\nSrc branch: All build attempts failed: Capture:\n#####  External Process Failed  #####\n`infer capture -R -- mvn compile`\n***STDOUT***\n\n```\n[INFO] Scanning for projects...\n[INFO] Inspecting build with total of 1 modules...\n[INFO] Installing Nexus Staging features:\n[INFO]   ... total of 1 executions of maven-deploy-plugin replaced with nexus-staging-maven-plugin\n[INFO]                                                                         \n[INFO] ------------------------------------------------------------------------\n[INFO] Building dynamic-spout 0.10-SNAPSHOT\n[INFO] ------------------------------------------------------------------------\n[INFO] \n[INFO] --- maven-checkstyle-plugin:2.17:checkstyle (checkstyle-validate) @ dynamic-spout ---\n[INFO] Starting audit...\nAudit done.\n[WARNING] Unable to locate Source XRef to link to - DISABLED\n[INFO] \n[INFO] --- maven-checkstyle-plugin:2.17:check (checkstyle-validate) @ dynamic-spout ---\n[INFO] Starting audit...\nAudit done.\n[INFO] \n[INFO] --- jacoco-maven-plugin:0.7.9:prepare-agent (pre-unit-test) @ dynamic-spout ---\n[INFO] Skipping JaCoCo execution because property jacoco.skip is set.\n[INFO] argLine set to empty\n[INFO] \n[INFO] --- maven-resources-plugin:2.6:resources (default-resources) @ dynamic-spout ---\n[INFO] Using 'UTF-8' encoding to copy filtered resources.\n[INFO] skip non existing resourceDirectory /tmp/inferring-571fbb675a45d41d/code/src/main/resources\n[INFO] \n[INFO] --- maven-compiler-plugin:3.6.1:compile (default-compile) @ dynamic-spout ---\n[INFO] Changes detected - recompiling the module!\n[INFO] Compiling 101 source files to /tmp/inferring-571fbb675a45d41d/code/target/classes\n[INFO] \n[INFO] --- license-maven-plugin:3.0:check (default) @ dynamic-spout ---\n[INFO] Checking licenses...\n[WARNING] Unknown file extension: /tmp/inferring-571fbb675a45d41d/code/somepatchforthecode.patch\n[WARNING] Unknown file extension: /tmp/inferring-571fbb675a45d41d/code/infer-out/results.db-shm\n[WARNING] Unknown file extension: /tmp/inferring-571fbb675a45d41d/code/infer-out/.global.tenv\n[WARNING] Unknown file extension: /tmp/inferring-571fbb675a45d41d/code/infer-out/logs\n[WARNING] Missing header in: /tmp/inferring-571fbb675a45d41d/code/infer_changed_files_index.txt\n[WARNING] Unknown file extension: /tmp/inferring-571fbb675a45d41d/code/infer-out/results.db\n[WARNING] Unknown file extension: /tmp/inferring-571fbb675a45d41d/code/pom.xml.infer-orig\n[WARNING] Unknown file extension: /tmp/inferring-571fbb675a45d41d/code/infer-out/results.db-wal\n[INFO] ------------------------------------------------------------------------\n[INFO] BUILD FAILURE\n[INFO] ------------------------------------------------------------------------\n[INFO] Total time: 14.204 s\n[INFO] Finished at: 2018-05-18T16:31:15-07:00\n[INFO] Final Memory: 35M/779M\n[INFO] ------------------------------------------------------------------------\n[ERROR] Failed to execute goal com.mycila:license-maven-plugin:3.0:check (default) on project dynamic-spout: Some files do not have the expected license header -> [Help 1]\n[ERROR] \n[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.\n[ERROR] Re-run Maven using the -X switch to enable full debug logging.\n[ERROR] \n[ERROR] For more information about the errors and possible solutions, please read the following articles:\n[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException\n\n```\n\n***STDERR***\n\n```\nCapturing in maven mode...\nUsage Error: *** Maven command failed:\n*** mvn compile -P infer-capture\n*** exited with code 1\n\n\nError while running epilogue \"restoring Maven's pom.xml to its original state\":\n(Unix.Unix_error \"No such file or directory\" rename\n \"((src /tmp/inferring-571fbb675a45d41d/code/pom.xml.infer-orig) (dst /tmp/inferring-571fbb675a45d41d/code/pom.xml))\").\nPowering through...\n\n```\n\n\n"}}
{"actor":{"display_login":"inferences"}}