Github user WegDamit commented on the issue:
https://github.com/apache/zeppelin/pull/1369
Those changes are only for dependencies given in the interpreter
definition, right?
Using a %spark.dep section in a notebook seems to uses a
SparkDependencyContext which does not inherit from
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
enabl
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1369
@doanduyhai Checked again and it works well! After removing the wrong
dependency, the error message doesn't show up anymore.
---
If your project is set up for it, you can reply to this email
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1369
@doanduyhai Yeah your screenshot is saying so. Let me test it again. Thanks
for your quick response!
---
If your project is set up for it, you can reply to this email and have your
reply appe
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 co
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1369
@doanduyhai So sorry for my late response ã
_ã
Tested again and checked
>when you navigate to different pages, the error popups are removed.
the above issue is gone! ð
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 by
appropri
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 :)
 {
var
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 `s
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 see
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 yo
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1369
I also checked the interpreter setting page like @astroshim did.
(edit interpreter -> add `one:two:1.1.1` to the dependency table -> save )
Two things are different with master br
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1369
@doanduyhai Could you check the infinitely downloading library problem(like
following screen shot)?
Strangely this problem happens all interpreter except spark. Is this
problem happen only
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 `shell`
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1369
Proxy repo tested and it works properly. :+1:
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1369
@doanduyhai BTW how come only spark interpreter shows `error message bar`?
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1369
@doanduyhai Good. I am going to check first commit(proxy) soon.
---
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 do
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 a
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 project
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1369
Wow very quick response!!:
I think set `combineDuplications` true is better but I don't have much
talent in UI.:)
---
If your project is set up for it, you can reply to this email and hav
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 getInterpreterSettings(
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1369
About the second commits, Is it intended behavior that the error message
bar is stacked whenever save the other interpreter setting?
