[GitHub] incubator-gearpump pull request: Gearpump 143

2016-05-31 Thread karol-brejna-i
Github user karol-brejna-i commented on the pull request: https://github.com/apache/incubator-gearpump/pull/30 ℵ₀ thanks for the patch! --- 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

[GitHub] incubator-gearpump pull request #44: GEARPUMP-164 configurable hbase user

2016-06-22 Thread karol-brejna-i
GitHub user karol-brejna-i opened a pull request: https://github.com/apache/incubator-gearpump/pull/44 GEARPUMP-164 configurable hbase user Simple change that allows configuring the user for connection to HBase. Tested on YARN deployment. You can merge this pull request

[GitHub] incubator-gearpump issue #11: fix #106 Gearpump Redis Integration

2016-06-16 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/11 I think I see the idea behind RedisSink and RedisStorage. Redis is advertised as "data structure store". It supports structures (like strings, lists, sets, h

[GitHub] incubator-gearpump issue #44: GEARPUMP-164 configurable hbase user

2016-06-23 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/44 I will clean up the commits and make sure scalastyle is ok with the PR. I will reissue new PR. --- If your project is set up for it, you can reply to this email and have your

[GitHub] incubator-gearpump pull request #44: GEARPUMP-164 configurable hbase user

2016-06-23 Thread karol-brejna-i
Github user karol-brejna-i closed the pull request at: https://github.com/apache/incubator-gearpump/pull/44 --- 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

[GitHub] incubator-gearpump issue #48: fix GEARPUMP161 application logs are located i...

2016-06-27 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/48 We met with @huafengw today. Something is still wrong. Huafeng is trying to peproduce. I am updating jira as new info arrives. Will update the pull request comments, too

[GitHub] incubator-gearpump issue #48: fix GEARPUMP161 application logs are located i...

2016-06-24 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/48 Sure, will do. I assigned the jira to me. --- 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

[GitHub] incubator-gearpump pull request: fix GEARPUMP-148, Document how to...

2016-05-23 Thread karol-brejna-i
Github user karol-brejna-i commented on a diff in the pull request: https://github.com/apache/incubator-gearpump/pull/22#discussion_r64182841 --- Diff: docs/deployment-ha.md --- @@ -38,6 +38,11 @@ In `conf/gear.conf`, For entry `gearpump.jarstore.rootpath`, please choose the s

[GitHub] incubator-gearpump pull request: gearpump-34 update developer docu...

2016-04-15 Thread karol-brejna-i
Github user karol-brejna-i commented on the pull request: https://github.com/apache/incubator-gearpump/pull/1#issuecomment-210462411 Small thing: IDE setup link (beginning of the page) doesn't work. It points to: https://github.com/whjiang/incubator-gearpump/blob/gearpump

[GitHub] incubator-gearpump pull request: gearpump-34 update developer docu...

2016-04-15 Thread karol-brejna-i
Github user karol-brejna-i commented on a diff in the pull request: https://github.com/apache/incubator-gearpump/pull/1#discussion_r59880173 --- Diff: CONTRIBUTING.md --- @@ -53,40 +137,242 @@ No work should ever be done in the forked master. Another way to do

[GitHub] incubator-gearpump pull request: gearpump-34 update developer docu...

2016-04-15 Thread karol-brejna-i
Github user karol-brejna-i commented on a diff in the pull request: https://github.com/apache/incubator-gearpump/pull/1#discussion_r59875167 --- Diff: CONTRIBUTING.md --- @@ -1,41 +1,125 @@ -This documents are for committers which have direct write permission to intel-hadoop

[GitHub] incubator-gearpump pull request: gearpump-34 update developer docu...

2016-04-15 Thread karol-brejna-i
Github user karol-brejna-i commented on a diff in the pull request: https://github.com/apache/incubator-gearpump/pull/1#discussion_r59873253 --- Diff: CONTRIBUTING.md --- @@ -53,40 +137,242 @@ No work should ever be done in the forked master. Another way to do

[GitHub] incubator-gearpump pull request: gearpump-34 update developer docu...

2016-04-15 Thread karol-brejna-i
Github user karol-brejna-i commented on a diff in the pull request: https://github.com/apache/incubator-gearpump/pull/1#discussion_r59873927 --- Diff: CONTRIBUTING.md --- @@ -53,40 +137,242 @@ No work should ever be done in the forked master. Another way to do

[GitHub] incubator-gearpump issue #48: fix GEARPUMP161 application logs are located i...

2016-06-28 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/48 @huafengw could you sum up what permissions (groups) does your user belong to? Maybe it is some kind of weird permission problem... --- If your project is set up for it, you can

[GitHub] incubator-gearpump pull request #141: [GEARPUMP-266] Add docker file

2017-02-08 Thread karol-brejna-i
Github user karol-brejna-i commented on a diff in the pull request: https://github.com/apache/incubator-gearpump/pull/141#discussion_r100010355 --- Diff: integrationtest/docker/Dockerfile --- @@ -0,0 +1,31 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] incubator-gearpump issue #141: [GEARPUMP-266] Add docker file

2017-02-08 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/141 Gladly ;-) I'll start something today and issue new PR if I find something. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] incubator-gearpump pull request #158: [GEARPUMP-267] Changing docker image f...

2017-02-22 Thread karol-brejna-i
GitHub user karol-brejna-i opened a pull request: https://github.com/apache/incubator-gearpump/pull/158 [GEARPUMP-267] Changing docker image for Kafka (spotify/kafka doesn't have tag for 0.8.x) Be sure to do all of the following to help us incorporate your contribution quickly

[GitHub] incubator-gearpump issue #158: [GEARPUMP-267] Changing docker image for Kafk...

2017-02-22 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/158 The good news is, Kafka "cluster" is up again (during integration tests). The bad news, there is some error running Kafka example. I found this in applic

[GitHub] incubator-gearpump issue #158: [GEARPUMP-267] Changing docker image for Kafk...

2017-02-24 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/158 I used Java 8 from Oracle, more precisely: ``` java version "1.8.0_121" Java(TM) SE Runtime Environment (build 1.8.0_121-b13) Java HotSpot(TM) 64-Bit Server

[GitHub] incubator-gearpump pull request #157: [GEARPUMP-236] Using new sbt-assembly ...

2017-02-21 Thread karol-brejna-i
GitHub user karol-brejna-i opened a pull request: https://github.com/apache/incubator-gearpump/pull/157 [GEARPUMP-236] Using new sbt-assembly version to fix java.lang.Verify… …Error Be sure to do all of the following to help us incorporate your contribution quickly

[GitHub] incubator-gearpump issue #157: [GEARPUMP-236] Using new sbt-assembly version...

2017-02-21 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/157 I saw them. I don't have a working hadoop cluster right now, so I had no way to confirm it work. I'll get myself a cluster soon, make the additional changes and test

[GitHub] incubator-gearpump issue #158: [GEARPUMP-267] Changing docker image for Kafk...

2017-02-22 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/158 On the other hand converting fron tihs date works on my dev machine: ``` import java.time.Instant type TimeStamp = Long val MIN_TIME_MILLIS: Long = Long.MinValue

[GitHub] incubator-gearpump issue #158: [GEARPUMP-267] Changing docker image for Kafk...

2017-02-22 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/158 Probably related to https://docs.oracle.com/javase/8/docs/api/java/time/Instant.html#toEpochMilli-- The logs show: `[INFO] [02/22/2017 10:25:34.395] [KafkaSource

[GitHub] incubator-gearpump pull request #162: [GEARPUMP-281] Using new version of do...

2017-02-28 Thread karol-brejna-i
Github user karol-brejna-i commented on a diff in the pull request: https://github.com/apache/incubator-gearpump/pull/162#discussion_r103432537 --- Diff: integrationtest/docker/kafka/README.md --- @@ -0,0 +1,89 @@ +Kafka in Docker +=== + +**This is a fork spotify

[GitHub] incubator-gearpump pull request #158: [GEARPUMP-267] Changing docker image f...

2017-02-28 Thread karol-brejna-i
Github user karol-brejna-i closed the pull request at: https://github.com/apache/incubator-gearpump/pull/158 --- 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

[GitHub] incubator-gearpump pull request #162: [GEARPUMP-281] Using new version of do...

2017-02-28 Thread karol-brejna-i
Github user karol-brejna-i commented on a diff in the pull request: https://github.com/apache/incubator-gearpump/pull/162#discussion_r103432682 --- Diff: integrationtest/docker/java/README.md --- @@ -0,0 +1,18 @@ +A minimalistic Oracle JDK 8 container on top of busybox

[GitHub] incubator-gearpump issue #158: [GEARPUMP-267] Changing docker image for Kafk...

2017-02-28 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/158 Now it's obsolete (implemented by two other PRs). Closing. --- 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

[GitHub] incubator-gearpump issue #90: fix GEARPUMP-144 make the user for HDFS and Ya...

2016-12-13 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/90 +1 works fine. also, in our new environments logs started to work properly (probably this and previous change helped). i closed the other ticket, too. --- If your project

[GitHub] incubator-gearpump pull request #175: Bump version to 0.8.4-SNAPSHOT

2017-04-19 Thread karol-brejna-i
GitHub user karol-brejna-i opened a pull request: https://github.com/apache/incubator-gearpump/pull/175 Bump version to 0.8.4-SNAPSHOT Be sure to do all of the following to help us incorporate your contribution quickly and easily: - [ ] Make sure the commit message

[GitHub] incubator-gearpump pull request #179: test

2017-05-05 Thread karol-brejna-i
GitHub user karol-brejna-i opened a pull request: https://github.com/apache/incubator-gearpump/pull/179 test Be sure to do all of the following to help us incorporate your contribution quickly and easily: - [ ] Make sure the commit message is formatted like

[GitHub] incubator-gearpump pull request #179: test

2017-05-05 Thread karol-brejna-i
Github user karol-brejna-i closed the pull request at: https://github.com/apache/incubator-gearpump/pull/179 --- 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

[GitHub] incubator-gearpump issue #230: [GEARPUMP-354] Fix the tracking url resolutio...

2017-10-11 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/230 @titikakatoo Please advise on quick/easy validation procedure. Also: Do you know any way to have kerberized yarn running in docker? ---

[GitHub] incubator-gearpump issue #233: [WIP] Fix integration test

2017-10-24 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/233 Even when providing proxy args for docker build: ``` docker build --no-cache --build-arg HTTP_PROXY=$http_proxy --build-arg HTTPS_PROXY=$https_proxy -t gearpump/gearpump

[GitHub] incubator-gearpump issue #233: [WIP] Fix integration test

2017-10-24 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/233 I've just confirmed. The problem persists, even when networking/proxy problems are overcomed: ``` $ docker build --no-cache -t gearpump/gearpump-launcher

[GitHub] incubator-gearpump issue #241: [WiP][GEARPUMP-368] Restore setting runtime e...

2018-04-27 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/241 Some IT tests failing. Trying to investigate. ---

[GitHub] incubator-gearpump issue #235: Use alpine-based image for gearpump-runner

2017-10-26 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/235 done! ---

[GitHub] incubator-gearpump pull request #235: Use alpine-based image for gearpump-ru...

2017-10-26 Thread karol-brejna-i
GitHub user karol-brejna-i opened a pull request: https://github.com/apache/incubator-gearpump/pull/235 Use alpine-based image for gearpump-runner Be sure to do all of the following to help us incorporate your contribution quickly and easily: - [ ] Make sure the commit

[GitHub] incubator-gearpump issue #233: [WIP] Fix integration test

2017-10-26 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/233 Docker image problem solved with https://github.com/apache/incubator-gearpump/pull/235. ---

[GitHub] incubator-gearpump issue #233: [GEARPUMP-361] Build gearpump docker image lo...

2017-10-27 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/233 The third problem - works for me on Ubuntu 16.04. ``` [DEBUG] [05:25:55.233] [Docker$] Container master0>> curl http://master0:8090/api/v1.0/master

[GitHub] incubator-gearpump issue #233: [GEARPUMP-361] Build gearpump docker image lo...

2017-10-27 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/233 ```[DEBUG] [13:07:44.774] [Docker$] Container master0>> curl http://master0:8090/api/v1.0/master [DEBUG] [13:07:44.774] [Docker$] INPUT==>> [DEBUG] [

[GitHub] incubator-gearpump pull request #233: [GEARPUMP-361] Build gearpump docker i...

2017-10-27 Thread karol-brejna-i
Github user karol-brejna-i commented on a diff in the pull request: https://github.com/apache/incubator-gearpump/pull/233#discussion_r147378048 --- Diff: integrationtest/docker/gearpump/Dockerfile --- @@ -18,7 +18,8 @@ FROM openjdk:8u131-jre-alpine # Install python

[GitHub] incubator-gearpump pull request #241: [GEARPUMP-368] Restore setting runtime...

2018-04-04 Thread karol-brejna-i
GitHub user karol-brejna-i opened a pull request: https://github.com/apache/incubator-gearpump/pull/241 [GEARPUMP-368] Restore setting runtime env Be sure to do all of the following to help us incorporate your contribution quickly and easily: - [x] Make sure the commit

[GitHub] incubator-gearpump issue #241: [GEARPUMP-368] Restore setting runtime env

2018-04-04 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/241 Two unrelated test fail: ``` [info] *** 2 TESTS FAILED *** [error] Failed: Total 35, Failed 2, Errors 0, Passed 33 [error] Failed tests: [error

[GitHub] incubator-gearpump issue #241: [WiP][GEARPUMP-368] Restore setting runtime e...

2018-04-23 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/241 One more thing to discuss: In yarn code we already have a trait named `Command`: experiments/yarn/src/main/scala/org/apache/gearpump/experiments/yarn/appmaster

[GitHub] incubator-gearpump issue #241: [WiP][GEARPUMP-368] Restore setting runtime e...

2018-04-23 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/241 I did search for AkkaApp usages and there are few groups of them: 1. examples (proper AkkaApp usage) 2. gearpupm-core (comands like submit, kill, etc. plus master and worker

[GitHub] incubator-gearpump issue #241: [WiP][GEARPUMP-368] Restore setting runtime e...

2018-04-23 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/241 I did the initial version. Will test today. ---