Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/2959
The fix is trivial, looks good to me
---
GitHub user doanduyhai reopened a pull request:
https://github.com/apache/zeppelin/pull/658
Switch to Java 8
### What is this PR for?
Java 7 is now officially no longer supported by Oracle and Java 8 has been
released quite a while. We should take the opportunity to switch
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/658
@mf69 done, PR re-opened
---
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1466
Let me redo the PR and merge with the above remarks included
---
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 doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1485
@ravwojdyla any update on this ? I face the same issue and need your PR to
go into trunk
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1466
Without feedback I'll push a new PR with suggested code refactor & merge it
in a few weeks @robertdkmarshall,
---
If your project is set up for it, you can reply to this email and
GitHub user doanduyhai opened a pull request:
https://github.com/apache/zeppelin/pull/1631
[ZEPPELIN-1535] Override JNA version with 4.2.0 for Power PC users
### What is this PR for?
For Power PC users, running embedded Cassandra fails because JNA version
4.0.0 is not working
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1485
I have the same compilation issue using `mvn clean compile`. After
excluding the same dependencies as this PR from `zeppelin-zengine/pom.xml` my
`mvn clean compile` is successful
---
If your
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1466
Ping @robertdkmarshall, any new from my code 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
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1369
OK I'll merge if no more remarks
---
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 fe
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1369
@AhyoungRyu
I followed your procedure but it works perfectly for me, see below
screenshot. Did you clean your browser cache when testing the new commit ?
Maybe there is some old JS
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1369
@AhyoungRyu @astroshim
Please tell me if it's ok so that I can merge. We need to merge this PR
before I can move on the refactoring task (of changing `synchronized` bloc
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/133
@Leemoonsoo
The fix is using pretty standard Scala (`object xxx extends Serializable`)
and not impacted by Scala 2.11 breaking changes
---
If your project is set up for it, you can
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/133
@Leemoonsoo
Yes we do need this patch because `DisplayUtils` has been disabled
temporarily because of the issue it causes. This patch fixes the issue and
re-enable `DisplayUtils
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/133
@corneadoug Comment updated with original issue description.
The JIRA is already created:
https://issues.apache.org/jira/browse/ZEPPELIN-127
---
If your project is set up for it, you
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1369
@astroshim @AhyoungRyu
Please see the final result, it should make you both happy :)
![fixed error
popups](https://cloud.githubusercontent.com/assets/1532977/19050426
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1369
@astroshim @AhyoungRyu
The wrong counting on error popup is now fixed. Every time you
* reload the interpreter page
* update an interpreter setting
all the error
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1369
@AhyoungRyu
> The ng-toast error message is still shown as @astroshim said in the above
comment. How about adding some timeout property to the message?
I knew that there i
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1473
Thank you @jongyoul it works like a charm ð
---
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 doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1473
@jongyoul I would like too but I don't have the commit right ...
---
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 pr
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1473
The code looks good 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 have this feature
enabled and
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/86
@benbromhead some remarks before rebasing
1) maybe upgrade C* version and Spark/Cassandra connector version
2) maybe use Achilles-embedded (look Cassandra interpreter unit tests
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/133
@corneadoug Yes we need it, it's a very convenient helper class to display
html/img ...
---
If your project is set up for it, you can reply to this email and have your
reply appear on G
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1439
Yes it's very clear. Indeed a bunch of internal features (Angular registry,
Callback registry ...) are exposed to end-users only via Spark interpreter. I
guess that each interp
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1439
Question, how interpreters other than Spark can leverage this new feature ?
I mean you added a new `registerCallback()` function for ZeppelinContext object
but this context is not exposed for
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1369
@astroshim
> the number seems doesn't match with count
What do you mean by this ? Which number did you expect to see ?
On the animated screencapture, t
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1369
@astroshim
The number you see in front of each error popup is normal, it comes from
the fact that now we set `combineDuplications = true` for **ngToast**. The
number corresponds to
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1369
@AhyoungRyu @astroshim Is the fix ok for you now ? The build is green
---
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 user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1369
@astroshim @AhyoungRyu
Ok I've found the issue, it's my bad:
```javascript
for (var setting = 0; setting < $scope.interpreterSettings.len
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1369
@astroshim @AhyoungRyu it tooks me hours but I have found the root cause of
the issue and it's pretty ugly. Ready to hear ?
1) First issue is we **do not** set the InterpreterSe
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1369
O I've found the issue, it's tricky.
I have added a non-existing dependency to the angular interpreter and I can
also see a spinning circle forever. With Web Dev Cons
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1369
Let me investigate the root cause, it should be some Javascript subtle bug
---
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 user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1369
@astroshim
> BTW how come only spark interpreter shows error message bar?
The error popup applies to all interpreters, try to add a fictive
dependency to `angular` or `sh
Github user doanduyhai closed the pull request at:
https://github.com/apache/zeppelin/pull/1369
---
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 the feature
GitHub user doanduyhai reopened a pull request:
https://github.com/apache/zeppelin/pull/1369
[ZEPPELIN-1376] Add proxy credentials for dependency repo for corporate
firewall use-cases
### What is this PR for?
When using Zeppelin behind corporate firewall, sometimes the
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/746
Thank you @felixcheung. Finally merged !
---
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
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1369
@astroshim Done, now the popup error is only displayed once even if you
update many other interpreters.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1369
Ok let me push a new commit with the `combineDuplications = true`
---
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 user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1369
@astroshim
It is explained above:
> The problem is that $route.reload(); in the success callback of
updateInterpreterSetting() will trigger init() then getInterpreterSetti
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/746
Ping @Leemoonsoo , can we merge this long-standing PR ? CI failures are
irrelevant as mentioned above
---
If your project is set up for it, you can reply to this email and have your
reply
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/746
Ok, there are 3 unrelated errors:
1) https://s3.amazonaws.com/archive.travis-ci.org/jobs/155941982/log.txt
```
[32mAngularElem[0m
[31m- should provide onclick method
Github user doanduyhai closed the pull request at:
https://github.com/apache/zeppelin/pull/1369
---
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 the feature
GitHub user doanduyhai reopened a pull request:
https://github.com/apache/zeppelin/pull/1369
[ZEPPELIN-1376] Add proxy credentials for dependency repo for corporate
firewall use-cases
### What is this PR for?
When using Zeppelin behind corporate firewall, sometimes the
GitHub user doanduyhai reopened a pull request:
https://github.com/apache/zeppelin/pull/746
[ZEPPELIN-699] Add new synchronous paragraph run REST API
### What is this PR for?
Right now, when calling the REST API
`http://:/api/notebook/job//` Zeppelin always
returns **OK** as
Github user doanduyhai closed the pull request at:
https://github.com/apache/zeppelin/pull/746
---
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 the feature
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1369
@jongyoul Sorry, it was a bad copy/paste between Github & JIRA, fixed now
---
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 doanduyhai reopened a pull request:
https://github.com/apache/zeppelin/pull/1369
[ZEPPELIN-1376]
### What is this PR for?
When using Zeppelin behind corporate firewall, sometimes the dependencies
download just fails silently. This PR has 2 objectives:
* add
Github user doanduyhai closed the pull request at:
https://github.com/apache/zeppelin/pull/1369
---
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 the feature
GitHub user doanduyhai reopened a pull request:
https://github.com/apache/zeppelin/pull/746
[ZEPPELIN-699] Add new synchronous paragraph run REST API
### What is this PR for?
Right now, when calling the REST API
`http://:/api/notebook/job//` Zeppelin always
returns **OK** as
Github user doanduyhai closed the pull request at:
https://github.com/apache/zeppelin/pull/746
---
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 the feature
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1376
@bzz
I don't know why the tests fail on this error message, because this is
indeed **a test which expects a failure** because `system.complex_table` does
not exist, see
GitHub user doanduyhai reopened a pull request:
https://github.com/apache/zeppelin/pull/746
[ZEPPELIN-699] Add new synchronous paragraph run REST API
### What is this PR for?
Right now, when calling the REST API
`http://:/api/notebook/job//` Zeppelin always
returns **OK** as
Github user doanduyhai closed the pull request at:
https://github.com/apache/zeppelin/pull/746
---
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 the feature
GitHub user doanduyhai reopened a pull request:
https://github.com/apache/zeppelin/pull/746
[ZEPPELIN-699] Add new synchronous paragraph run REST API
### What is this PR for?
Right now, when calling the REST API
`http://:/api/notebook/job//` Zeppelin always
returns **OK** as
Github user doanduyhai closed the pull request at:
https://github.com/apache/zeppelin/pull/746
---
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 the feature
Github user doanduyhai closed the pull request at:
https://github.com/apache/zeppelin/pull/746
---
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 the feature
GitHub user doanduyhai reopened a pull request:
https://github.com/apache/zeppelin/pull/746
[ZEPPELIN-699] Add new synchronous paragraph run REST API
### What is this PR for?
Right now, when calling the REST API
`http://:/api/notebook/job//` Zeppelin always
returns **OK** as
Github user doanduyhai closed the pull request at:
https://github.com/apache/zeppelin/pull/746
---
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 the feature
GitHub user doanduyhai reopened a pull request:
https://github.com/apache/zeppelin/pull/746
[ZEPPELIN-699] Add new synchronous paragraph run REST API
### What is this PR for?
Right now, when calling the REST API
`http://:/api/notebook/job//` Zeppelin always
returns **OK** as
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1369
@felixcheung
That's the main issue I'm facing now, how to **automate** test with
corporate firewall ... I'm going to ask someone to test the this PR at his
place (he
GitHub user doanduyhai opened a pull request:
https://github.com/apache/zeppelin/pull/1369
[ZEPPELIN-1376]
### What is this PR for?
When using Zeppelin behind corporate firewall, sometimes the dependencies
download just fails silently. This PR has 2 objectives:
* add
Github user doanduyhai closed the pull request at:
https://github.com/apache/zeppelin/pull/746
---
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 the feature
GitHub user doanduyhai reopened a pull request:
https://github.com/apache/zeppelin/pull/746
[ZEPPELIN-699] Add new synchronous paragraph run REST API
### What is this PR for?
Right now, when calling the REST API
`http://:/api/notebook/job//` Zeppelin always
returns **OK** as
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1262
By the way, those issues are all related to the fact that we are running
our software factory on a shared/mutualized infrastructure (Travis). I have the
same issue of random failures due to
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1262
@jongyoul The only way to avoid this failure is to block the unit tests
until the embedded Cassandra server is up, which can take time and sometimes
end in a dead-lock situation
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/1262
@jongyoul
`caused by: com.datastax.driver.core.exceptions.NoHostAvailableException:
All host(s) tried for query failed (tried: localhost/127.0.0.1:9107
GitHub user doanduyhai reopened a pull request:
https://github.com/apache/zeppelin/pull/746
[ZEPPELIN-699] Add new synchronous paragraph run REST API
### What is this PR for?
Right now, when calling the REST API
`http://:/api/notebook/job//` Zeppelin always
returns **OK** as
Github user doanduyhai closed the pull request at:
https://github.com/apache/zeppelin/pull/746
---
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 the feature
GitHub user doanduyhai reopened a pull request:
https://github.com/apache/zeppelin/pull/746
[ZEPPELIN-699] Add new synchronous paragraph run REST API
### What is this PR for?
Right now, when calling the REST API
`http://:/api/notebook/job//` Zeppelin always
returns **OK** as
Github user doanduyhai closed the pull request at:
https://github.com/apache/zeppelin/pull/746
---
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 the feature
Github user doanduyhai closed the pull request at:
https://github.com/apache/zeppelin/pull/746
---
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 the feature
GitHub user doanduyhai reopened a pull request:
https://github.com/apache/zeppelin/pull/746
[ZEPPELIN-699] Add new synchronous paragraph run REST API
### What is this PR for?
Right now, when calling the REST API
`http://:/api/notebook/job//` Zeppelin always
returns **OK** as
Github user doanduyhai commented on the issue:
https://github.com/apache/zeppelin/pull/746
Rebased from master
---
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 doanduyhai commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/746
@felixcheung CI green, you can resume code 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
Github user doanduyhai closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/746
---
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 user doanduyhai reopened a pull request:
https://github.com/apache/incubator-zeppelin/pull/746
[ZEPPELIN-699] Add new synchronous paragraph run REST API
### What is this PR for?
Right now, when calling the REST API
`http://:/api/notebook/job//` Zeppelin always
returns
Github user doanduyhai closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/746
---
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 user doanduyhai reopened a pull request:
https://github.com/apache/incubator-zeppelin/pull/746
[ZEPPELIN-699] Add new synchronous paragraph run REST API
### What is this PR for?
Right now, when calling the REST API
`http://:/api/notebook/job//` Zeppelin always
returns
Github user doanduyhai closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/950
---
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 user doanduyhai reopened a pull request:
https://github.com/apache/incubator-zeppelin/pull/950
[ZEPPELIN-901] Cassandra interpreter V3
### What is this PR for?
**Cassandra** interpreter V3:
* Update documentation
* Update interactive documentation
* Add
GitHub user doanduyhai reopened a pull request:
https://github.com/apache/incubator-zeppelin/pull/950
[ZEPPELIN-901] Cassandra interpreter V3
### What is this PR for?
**Cassandra** interpreter V3:
* Update documentation
* Update interactive documentation
* Add
Github user doanduyhai closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/950
---
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 user doanduyhai closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/746
---
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 user doanduyhai reopened a pull request:
https://github.com/apache/incubator-zeppelin/pull/746
[ZEPPELIN-699] Add new synchronous paragraph run REST API
### What is this PR for?
Right now, when calling the REST API
`http://:/api/notebook/job//` Zeppelin always
returns
Github user doanduyhai commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/746
Hello @bzz, I've rebased from the latest master and my PR always fail with
this issue:
```
12:28:28,504 INFO org.apache.zeppelin.AbstractZeppelinIT:59 - Fin
Github user doanduyhai closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/746
---
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 user doanduyhai reopened a pull request:
https://github.com/apache/incubator-zeppelin/pull/746
[ZEPPELIN-699] Add new synchronous paragraph run REST API
### What is this PR for?
Right now, when calling the REST API
`http://:/api/notebook/job//` Zeppelin always
returns
Github user doanduyhai commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/746
@felixcheung Done, I've added 2 new screen captures of the API
documentation update
---
If your project is set up for it, you can reply to this email and have your
reply appe
GitHub user doanduyhai reopened a pull request:
https://github.com/apache/incubator-zeppelin/pull/950
[ZEPPELIN-901] Cassandra interpreter V3
### What is this PR for?
**Cassandra** interpreter V3:
* Update documentation
* Update interactive documentation
* Add
Github user doanduyhai closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/950
---
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 user doanduyhai commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/950
@AhyoungRyu PR updated with your remarks
---
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 doanduyhai commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/746
@felixcheung I've updated the PR to create a new API as suggested by
@Leemoonsoo : `/run/{noteId}/{paragraphId}`.
All my apologies for the long delay, I was lately busy
GitHub user doanduyhai opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/950
[ZEPPELIN-901] Cassandra interpreter V3
### What is this PR for?
**Cassandra** interpreter V3:
* Update documentation
* Update interactive documentation
* Add
93 matches
Mail list logo