[GitHub] rdhabalia opened a new pull request #2548: Stop all functions gracefully on closing worker-service

2018-09-09 Thread GitBox
rdhabalia opened a new pull request #2548: Stop all functions gracefully on closing worker-service URL: https://github.com/apache/incubator-pulsar/pull/2548 ### Motivation Right now, if stopping

[GitHub] YuiFn closed pull request #2547: Merge pull request #1 from apache/master

2018-09-09 Thread GitBox
YuiFn closed pull request #2547: Merge pull request #1 from apache/master URL: https://github.com/apache/incubator-pulsar/pull/2547 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] YuiFn opened a new pull request #2547: Merge pull request #1 from apache/master

2018-09-09 Thread GitBox
YuiFn opened a new pull request #2547: Merge pull request #1 from apache/master URL: https://github.com/apache/incubator-pulsar/pull/2547 merge remote ### Motivation Explain here the context, and why you're making that change. What is the problem you're trying to solve.

[GitHub] sijie commented on issue #2500: [schema] add schemas for primtive types

2018-09-09 Thread GitBox
sijie commented on issue #2500: [schema] add schemas for primtive types URL: https://github.com/apache/incubator-pulsar/pull/2500#issuecomment-419754258 run java8 tests run integration tests This is an automated message

[incubator-pulsar] branch master updated: [documentation] Add section about licensing at contribution guide (#2511)

2018-09-09 Thread sijie
This is an automated email from the ASF dual-hosted git repository. sijie pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git The following commit(s) were added to refs/heads/master by this push: new 3e9a445 [documentation] Add section

[GitHub] sijie closed pull request #2511: [documentation] Add section about licensing at contribution guide

2018-09-09 Thread GitBox
sijie closed pull request #2511: [documentation] Add section about licensing at contribution guide URL: https://github.com/apache/incubator-pulsar/pull/2511 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[incubator-pulsar] branch master updated: Added cmake install for lib and headers (#2529)

2018-09-09 Thread sijie
This is an automated email from the ASF dual-hosted git repository. sijie pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git The following commit(s) were added to refs/heads/master by this push: new 1c72ac3 Added cmake install for lib

[GitHub] sijie commented on issue #2545: [docker] introduce a pulsar standalone image

2018-09-09 Thread GitBox
sijie commented on issue #2545: [docker] introduce a pulsar standalone image URL: https://github.com/apache/incubator-pulsar/pull/2545#issuecomment-419746688 Sure. We can reorganize modules to do that. but it will take some time to make sure it doesn’t break others like CI, release process

[GitHub] kramasamy commented on issue #2545: [docker] introduce a pulsar standalone image

2018-09-09 Thread GitBox
kramasamy commented on issue #2545: [docker] introduce a pulsar standalone image URL: https://github.com/apache/incubator-pulsar/pull/2545#issuecomment-419744589 Another options is move the dockerfile to the top. Duplicating code and config will be difficult to manage as time goes on

[GitHub] sijie commented on issue #2545: [docker] introduce a pulsar standalone image

2018-09-09 Thread GitBox
sijie commented on issue #2545: [docker] introduce a pulsar standalone image URL: https://github.com/apache/incubator-pulsar/pull/2545#issuecomment-419743264 you can point to a location of a dockerfile. But in the dockerfile defintion, you can’t access files not in the same dir as the

[GitHub] cckellogg edited a comment on issue #2545: [docker] introduce a pulsar standalone image

2018-09-09 Thread GitBox
cckellogg edited a comment on issue #2545: [docker] introduce a pulsar standalone image URL: https://github.com/apache/incubator-pulsar/pull/2545#issuecomment-419742245 Is that a maven issue? There is a way to point to a specific file when doing a docker build "$ docker build -f

[GitHub] cckellogg commented on issue #2545: [docker] introduce a pulsar standalone image

2018-09-09 Thread GitBox
cckellogg commented on issue #2545: [docker] introduce a pulsar standalone image URL: https://github.com/apache/incubator-pulsar/pull/2545#issuecomment-419742245 Is that a maven issue? There is a way to point to a specific file when do a docker build "$ docker build -f

[GitHub] sijie commented on issue #2545: [docker] introduce a pulsar standalone image

2018-09-09 Thread GitBox
sijie commented on issue #2545: [docker] introduce a pulsar standalone image URL: https://github.com/apache/incubator-pulsar/pull/2545#issuecomment-419741657 We can’t do it, because docker doesn’t allow accessing files outside the dockerfile directory. To deliver this ASAP, we have to copy

[GitHub] kramasamy commented on issue #2545: [docker] introduce a pulsar standalone image

2018-09-09 Thread GitBox
kramasamy commented on issue #2545: [docker] introduce a pulsar standalone image URL: https://github.com/apache/incubator-pulsar/pull/2545#issuecomment-419741204 @sijie @merlimat - these code seems to be copy of the code under /dashboard. This is not a clean way to do this - since any

[GitHub] sijie commented on issue #2545: [docker] introduce a pulsar standalone image

2018-09-09 Thread GitBox
sijie commented on issue #2545: [docker] introduce a pulsar standalone image URL: https://github.com/apache/incubator-pulsar/pull/2545#issuecomment-419740749 @cckellogg not sure if we can easily change to sqlite. It is probably ok, however I would not suggest changing it right now. Because

[GitHub] david-streamlio commented on issue #2546: Elastic connector

2018-09-09 Thread GitBox
david-streamlio commented on issue #2546: Elastic connector URL: https://github.com/apache/incubator-pulsar/pull/2546#issuecomment-419734115 How do I view the actual cause of the error.. org.apache.pulsar.tests.integration.docker.ContainerExecException: /pulsar/bin/pulsar-admin sink

[GitHub] srkukarni commented on issue #2546: Elastic connector

2018-09-09 Thread GitBox
srkukarni commented on issue #2546: Elastic connector URL: https://github.com/apache/incubator-pulsar/pull/2546#issuecomment-419724450 run integration tests This is an automated message from the Apache Git Service. To respond