Github user iemejia commented on the issue:
https://github.com/apache/flink/pull/2176
Great, thanks for your review.
---
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
Github user aljoscha commented on the issue:
https://github.com/apache/flink/pull/2176
That's great to hear! I'll write something on the Beam ML thread.
---
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 user iemejia commented on the issue:
https://github.com/apache/flink/pull/2176
This should be ok now. In further commits I expect to fix the daemon thing
+ maybe add a HA version using zookeeper of the docker-compose file.
One more question aljoscha, I intend to add the
Github user aljoscha commented on the issue:
https://github.com/apache/flink/pull/2176
I had two more comments about the README but after that it should be good
to merge.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user aljoscha commented on the issue:
https://github.com/apache/flink/pull/2176
You were right, I did exactly the same thing I did on OS X on a new Ubuntu
16.04 installation and it worked. ð
---
If your project is set up for it, you can reply to this email and have your
Github user aljoscha commented on the issue:
https://github.com/apache/flink/pull/2176
Yes, this is exactly what I was trying on OS X. I'm quickly setting up a
ubuntu VM to see if it works there.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user iemejia commented on the issue:
https://github.com/apache/flink/pull/2176
Hi, I tested it with the basic word count and with the beam pipeline
example that @ecesena put for his flink/beam demo.
I don't know if your case is maybe because you are running on docker
Github user aljoscha commented on the issue:
https://github.com/apache/flink/pull/2176
One last thing I would like to try is running a job from an existing Flink
installation using `$FLINK_HOME/bin/flink run -m `
as described in the README.
I suspect it has something to do
Github user iemejia commented on the issue:
https://github.com/apache/flink/pull/2176
Awasome, thanks @aljoscha, let's merge !
---
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
Github user aljoscha commented on the issue:
https://github.com/apache/flink/pull/2176
Re 1. I think this should be fine. There is some dynamic code generation
but this uses Janino as a library so that shouldn't be a problem.
Re 2. I'm not aware of any but it should be easy
Github user iemejia commented on the issue:
https://github.com/apache/flink/pull/2176
Nice, I just fixed as you suggested. I have three questions:
1. This container is based on the Java JRE (to keep it small), Does Flink
in any part do some magic that requires a full JDK
Github user aljoscha commented on the issue:
https://github.com/apache/flink/pull/2176
Nice work! I didn't know how to use docker but I managed to set it up and
use the new version on OS X without a problem. So it seems to work well, and
the code is a lot simpler and the image is
Github user iemejia commented on the issue:
https://github.com/apache/flink/pull/2176
Sorry I had to rebase my previous PR but this is the definitive one.
---
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 user iemejia commented on the issue:
https://github.com/apache/flink/pull/2176
The docker images script was simplified and the image size was reduced.
Previous image:
flink latest 6475add651c7 24 minutes ago 711.6 MB
Image after FLINK-4118
flink latest
14 matches
Mail list logo