GitHub PullRequestReviewCommentEvent at 2014-04-23 14:49:58
Repository: stratosphere
Actor: fhueske
Payload: {"comment":{"url":"https://api.github.com/repos/stratosphere/stratosphere/pulls/comments/11903678","id":11903678,"diff_hunk":"@@ -0,0 +1,44 @@\n+/***********************************************************************************************************************\n+ *\n+ * Copyright (C) 2010-2013 by the Stratosphere project (http://stratosphere.eu)\n+ *\n+ * Licensed under the Apache License, Version 2.0 (the \"License\"); you may not use this file except in compliance with\n+ * the License. You may obtain a copy of the License at\n+ *\n+ *     http://www.apache.org/licenses/LICENSE-2.0\n+ *\n+ * Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on\n+ * an \"AS IS\" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the\n+ * specific language governing permissions and limitations under the License.\n+ *\n+ **********************************************************************************************************************/\n+package eu.stratosphere.api.java.operators.translation;\n+\n+import eu.stratosphere.api.java.tuple.Tuple2;\n+import eu.stratosphere.util.Collector;\n+\n+/**\n+ * Needed to wrap tuples to Tuple2<key, value> pairs for combine method of group reduce with key selector function\n+ */\n+public class TupleWrappingCollector<K, IN> implements Collector<IN> {\n+\n+\tprivate K key;\n+\t\n+\tprivate Collector<Tuple2<K, IN>> outerCollector;\n+\t\n+\tpublic TupleWrappingCollector(K key, Collector<Tuple2<K, IN>> outerCollector) {","path":"stratosphere-java/src/main/java/eu/stratosphere/api/java/operators/translation/TupleWrappingCollector.java","position":29,"original_position":29,"commit_id":"4951914d2727cb42c431242ccb8d3e03983ca83f","original_commit_id":"4951914d2727cb42c431242ccb8d3e03983ca83f","user":{"login":"fhueske","id":2388347,"avatar_url":"https://avatars.githubusercontent.com/u/2388347?","gravatar_id":"3be795416981e6f1beb30dd45a91a4aa","url":"https://api.github.com/users/fhueske","html_url":"https://github.com/fhueske","followers_url":"https://api.github.com/users/fhueske/followers","following_url":"https://api.github.com/users/fhueske/following{/other_user}","gists_url":"https://api.github.com/users/fhueske/gists{/gist_id}","starred_url":"https://api.github.com/users/fhueske/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fhueske/subscriptions","organizations_url":"https://api.github.com/users/fhueske/orgs","repos_url":"https://api.github.com/users/fhueske/repos","events_url":"https://api.github.com/users/fhueske/events{/privacy}","received_events_url":"https://api.github.com/users/fhueske/received_events","type":"User","site_admin":false},"body":"The TupleWrappingCollector should \r\n- also have an (empty) default constructor.\r\n- allow to reset the key and wrapped collector.","created_at":"2014-04-23T14:49:58Z","updated_at":"2014-04-23T14:49:58Z","html_url":"https://github.com/stratosphere/stratosphere/pull/713#discussion_r11903678","pull_request_url":"https://api.github.com/repos/stratosphere/stratosphere/pulls/713","_links":{"self":{"href":"https://api.github.com/repos/stratosphere/stratosphere/pulls/comments/11903678"},"html":{"href":"https://github.com/stratosphere/stratosphere/pull/713#discussion_r11903678"},"pull_request":{"href":"https://api.github.com/repos/stratosphere/stratosphere/pulls/713"}}}}