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 have
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
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 feature
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` blocks
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 :)
 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 GitHub
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 interpreter
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
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, the co
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
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.length;
sett
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 InterpreterSetting
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 Console, I can
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
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
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
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
@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 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
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
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
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 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
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 below
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
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
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
@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 has a corporate
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
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
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 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
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
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/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
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
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
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
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 appear
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
67 matches
Mail list logo
