Github user ravwojdyla commented on the issue:
https://github.com/apache/zeppelin/pull/1471
@AhyoungRyu sorry I missed it. Will go ahead and add all the licenses.
Thanks!
---
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 ravwojdyla commented on the issue:
https://github.com/apache/zeppelin/pull/1471
ping @AhyoungRyu @zjffdu
---
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
Github user ravwojdyla commented on the issue:
https://github.com/apache/zeppelin/pull/1471
@AhyoungRyu updated licenses. The travis failures do not seems to be
related could you please restart the travis jobs?
---
If your project is set up for it, you can reply to this email
Github user ravwojdyla commented on the issue:
https://github.com/apache/zeppelin/pull/1471
The tests are failing for:
```
Tests run: 9, Failures: 2, Errors: 0, Skipped: 0, Time elapsed: 119.221 sec
<<< FAILURE! - in org.apache.zeppelin.integration.Paragraph
GitHub user ravwojdyla opened a pull request:
https://github.com/apache/zeppelin/pull/1471
Closes ZEPPELIN-1505: Add Scio interpreter
### What is this PR for?
Closes #ZEPPELIN-1505. Adds Scio interpreter. Scio is a Scala DSL on top of
Dataflow/Beam.
### What type of PR
Github user ravwojdyla commented on the issue:
https://github.com/apache/zeppelin/pull/1471
@AhyoungRyu @zjffdu thanks for prompt review. I have some questions in the
comments, plus fixes coming in. I have also changed the topic of the PR.
---
If your project is set up for it, you
Github user ravwojdyla commented on the issue:
https://github.com/apache/zeppelin/pull/1471
@AhyoungRyu @zjffdu are there any more comment, how do you want to proceed?
the error on travis does not seems to be related?
---
If your project is set up for it, you can reply to this email
Github user ravwojdyla commented on the issue:
https://github.com/apache/zeppelin/pull/1471
@AhyoungRyu @zjffdu could you please review when your have time. all the
todos are âï¸
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user ravwojdyla commented on the issue:
https://github.com/apache/zeppelin/pull/1471
ping @felixcheung
---
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
Github user ravwojdyla commented on the issue:
https://github.com/apache/zeppelin/pull/1471
Rebased to solve conflict in
`zeppelin-distribution/src/bin_license/LICENSE`. Are there any other comments?
---
If your project is set up for it, you can reply to this email and have your
Github user ravwojdyla commented on the issue:
https://github.com/apache/zeppelin/pull/1471
Those failures on travis seem unrelated - restarting build.
---
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 user ravwojdyla commented on the issue:
https://github.com/apache/zeppelin/pull/1471
@felixcheung well - yes and no. scio is currently being contributed to
beam, but current implementation still uses dataflow sdk. maybe we can
consolidate it once scio migration to beam is done
Github user ravwojdyla commented on the issue:
https://github.com/apache/zeppelin/pull/1471
ping @felixcheung
---
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
Github user ravwojdyla commented on the issue:
https://github.com/apache/zeppelin/pull/1471
@felixcheung I see, thanks for help, is 95dd9bb enough?
---
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 user ravwojdyla commented on the issue:
https://github.com/apache/zeppelin/pull/1471
@felixcheung after going through the `BeamInterpreter`, I don't see
immediate technical advantage of sharing the group, current implementation of
`BeamInterpreter`, is actually pure Java
Github user ravwojdyla commented on the issue:
https://github.com/apache/zeppelin/pull/1471
ping @Leemoonsoo @felixcheung
---
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 ravwojdyla opened a pull request:
https://github.com/apache/zeppelin/pull/1633
Fix formatting/typos/style in scio doc
### What is this PR for?
Fix formatting/typos/style in scio doc
### What type of PR is it?
Documentation
You can merge this pull
Github user ravwojdyla commented on the issue:
https://github.com/apache/zeppelin/pull/1633
Is jira ticket needed for this?
---
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 ravwojdyla commented on the issue:
https://github.com/apache/zeppelin/pull/1471
@felixcheung correct - i missed it - updated doc/examples.
---
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 user ravwojdyla commented on the issue:
https://github.com/apache/zeppelin/pull/1471
@nevillelyh (the creator of scio) - could you please 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
Github user ravwojdyla commented on the issue:
https://github.com/apache/zeppelin/pull/1471
@AhyoungRyu @zjffdu code was reviewed by @nevillelyh, build looks green.
Any other comments?
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user ravwojdyla commented on the issue:
https://github.com/apache/zeppelin/pull/1471
@zjffdu @AhyoungRyu are there any other comments?
---
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 user ravwojdyla opened a pull request:
https://github.com/apache/zeppelin/pull/1755
[ZEPPELIN-1801] To force end table - append %text
### What is this PR for?
It seems there is a bug in scio introduced by recent changes. Table
rendered from scio helper methods would
Github user ravwojdyla commented on the issue:
https://github.com/apache/zeppelin/pull/1754
@felixcheung thanks for the review - tests fail, but I doubt it's due to
this change.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
GitHub user ravwojdyla opened a pull request:
https://github.com/apache/zeppelin/pull/1754
[ZEPPELIN-1800] Update scio doc - fix type $ should be %
### What is this PR for?
Update scio doc - fix type $ should be %
### What type of PR is it?
Documentation
25 matches
Mail list logo
