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

2018-04-29 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/241 @manuzhang Please, take a look. ---

[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 #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. ---

[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 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 #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 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 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 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 #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-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-lau

[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 #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 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, or

[GitHub] incubator-gearpump pull request #177: [GEARPUMP-297] Make urls relative

2017-04-26 Thread karol-brejna-i
GitHub user karol-brejna-i opened a pull request: https://github.com/apache/incubator-gearpump/pull/177 [GEARPUMP-297] Make urls relative Be sure to do all of the following to help us incorporate your contribution quickly and easily: - [x] Make sure the commit message

[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 is

[GitHub] incubator-gearpump pull request #173: [GEARPUMP-294] Create RC1 for 0.8.3 re...

2017-04-05 Thread karol-brejna-i
GitHub user karol-brejna-i opened a pull request: https://github.com/apache/incubator-gearpump/pull/173 [GEARPUMP-294] Create RC1 for 0.8.3 release Be sure to do all of the following to help us incorporate your contribution quickly and easily: - [`` ] Make sure the

[GitHub] incubator-gearpump issue #162: [GEARPUMP-281] Using new version of docker im...

2017-03-01 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/162 @huafengw no problem. Let me get back to you soon. --- 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

[GitHub] incubator-gearpump issue #162: [GEARPUMP-281] Using new version of docker im...

2017-03-01 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/162 The change you suggested works fine on my ubuntu 14.04: ``` [info] Run completed in 25 minutes, 45 seconds. [info] Total number of tests run: 66 [info] Suites

[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_r103432650 --- Diff: integrationtest/docker/kafka/LICENSE --- @@ -0,0 +1,201 @@ +Apache License --- End diff -- added licence header

[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 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, or

[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 #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_r103432669 --- Diff: integrationtest/docker/README.md --- @@ -1,28 +1,23 @@ -# Gearpump Launcher Docker Image - -The image helps

[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

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

2017-02-27 Thread karol-brejna-i
GitHub user karol-brejna-i opened a pull request: https://github.com/apache/incubator-gearpump/pull/162 [GEARPUMP-281] Using new version of docker images for integration tests Be sure to do all of the following to help us incorporate your contribution quickly and easily

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

2017-02-27 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/158 I've collected docker definitions in one place (integrationtest/docker), upgraded java version for gearpump-launcher, switched to "our" kafka image (0.8.2.1): htt

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

2017-02-27 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/158 gearump-launcher that we use in integration tests uses errordeveloper/oracle-jre image that has JDK 8u45-b14. I updated JRE image and will use it in gearpump-launcher to see if

[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 issue #157: [GEARPUMP-236] Using new sbt-assembly version...

2017-02-23 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/157 Removed novverify from yarn related code. --- 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

[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 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 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 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 qu

[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 a

[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 #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 G

[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 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_r100010229 --- Diff: integrationtest/docker/Dockerfile --- @@ -0,0 +1,31 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[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 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 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 not

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

2016-06-23 Thread karol-brejna-i
GitHub user karol-brejna-i opened a pull request: https://github.com/apache/incubator-gearpump/pull/49 GEARPUMP-164 configurable hbase user Simple change that allows configuring the user for connection to HBase. Tested on YARN deployment. (PR replacing messy https

[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, or if

[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 issue #11: fix #106 Gearpump Redis Integration

2016-06-22 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/11 @darionyaphet > Support more command such as : time to live , exist and delete . This is the reason I tend to agree @manuzhang. Let's have the "basic&

[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 into a

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

2016-06-20 Thread karol-brejna-i
Github user karol-brejna-i commented on the issue: https://github.com/apache/incubator-gearpump/pull/11 We are developing under 'experiments' here. I'd be happy to have something decent here. Maybe not full-blown, but functional. Can we sum up what we still need

[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 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: 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 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 this is to

[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_r59879821 --- Diff: CONTRIBUTING.md --- @@ -53,40 +137,242 @@ No work should ever be done in the forked master. Another way to do this is to

[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 this is to

[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 this is to

[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_r59879526 --- Diff: CONTRIBUTING.md --- @@ -53,40 +137,242 @@ No work should ever be done in the forked master. Another way to do this is to

[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_r59873136 --- 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 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/gea