Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1394
Ah I missed the change. LGTM
---
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 corneadoug commented on the issue:
https://github.com/apache/zeppelin/pull/1265
Since I don't know much about how the interpreter part works, Its still
quite confusing.
And I think that having both checkbox and a dropdown might not be helping.
First of all, is
Github user lresende commented on the issue:
https://github.com/apache/zeppelin/pull/1400
This should go for both master and 0.6x release.
---
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 lresende commented on the issue:
https://github.com/apache/zeppelin/pull/1400
@bzz @Leemoonsoo Please review, this is more like a hotfix for the 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
GitHub user lresende opened a pull request:
https://github.com/apache/zeppelin/pull/1400
[ZEPPELIN-1407] Fix Scala 2.11 build
### What is this PR for?
Avoid activating the Scala 2.10 profile when building for Scala 2.11
### What type of PR is it?
[Bug Fix]
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1394
Yes, I did both, rename the directory and
https://github.com/prabhjyotsingh/zeppelin/blob/5993506b77be84d71ab730c2965572edd73e870e/notebook/2BWJFTXKJ/note.json#L1041
---
If your project
Github user lresende commented on the issue:
https://github.com/apache/zeppelin/pull/1376
Now the issues seems to be related to build timeout ?
No output has been received in the last 10 minutes, this potentially
indicates a stalled build or something wrong with the build itself.
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1334
I think having it as it is now sounds as a good start. It can be always
enhanced it later on, based on user feedback.
Changes look great to me, thank you!
If that is ok - please let
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1265
Concerning those modes, AFAIK, before Zeppelin 0.6.0, Zeppelin has only
`shared` mode, but most of users want to run a interpreter with their own,
Zeppelin add two modes called `scoped` and
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1333
Very similar test failure as we discussed above happened in un-related
#1363
[here](https://s3.amazonaws.com/archive.travis-ci.org/jobs/155836675/log.txt)
```
Failed tests:
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1389
---
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 is
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1333
@zjffdu so there must be some state that persists between the runs.
@Leemoonsoo do you have any idea what that state might be that affects
`InterpreterFactoryTest `? There seems to be
Github user lresende commented on the issue:
https://github.com/apache/zeppelin/pull/1376
@bzz Based on the jira, the exception is causing some confusion with users,
so I would merge, but it's up to the community, I am fine if there are
objections with this change.
And let
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1371
Looks great to me, thank you @Peilin-Yang !
@corneadoug what do you think?
Let's merge to master, if there is no further discussion.
---
If your project is set up for it, you can
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1376
@lresende shall we merge it or do you find @zjffdu suggestion to work and
we can close it for now?
Thank you @doanduyhai ! Let's try to close\open it to trigger the CI and
see if the failure
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1302
Looks great to me, thank you @astroshim !
On @corneadoug point - it should be possible to animate the icon button \w
arrows to be i.e rotating while "restart is in progress" so it stops
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1378
I see, indeed we could change the default to always show dataframe index,
if you think it's more consistent with Python behavior, but let's add a flag
anyway, so later, in case we decided to switch
Luciano Resende created ZEPPELIN-1407:
-
Summary: Scala 2.11 build fails with empty maven repository
Key: ZEPPELIN-1407
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1407
Project: Zeppelin
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1394
Shell we update
https://github.com/apache/zeppelin/blob/master/notebook/r/note.json#L1011, too?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1384
I'v restored markdown4j parser and add `markdown.parser.type` property to
`md` group. Pegdown parser will be used only if user specify `pegdown` property
value.
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1384
@Leemoonsoo
I think **option 2** (Adding a property) is a better solution. because
1) Adding a new interpreter just for support another markdown parser might
be confusing to
Github user echarles commented on the issue:
https://github.com/apache/zeppelin/pull/1385
@jongyoul `ZeppelinConfiguration.create()` is now used and method signature
are now like before.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user echarles commented on the issue:
https://github.com/apache/zeppelin/pull/1390
@zjffdu Correct, this PR also implements ZEPPELIN-1338 User level
interpreter setting (I have updated the title and description on this PR).
@Leemoonsoo Will check and adapt if needed
Github user echarles commented on the issue:
https://github.com/apache/zeppelin/pull/1394
@prabhjyotsingh yes, assumption is correct. At the time of the R
interpreter dev, I was used to manually rename the note folder and edit the
json to set the id the same value.
This PR
GitHub user rajarajan-g opened a pull request:
https://github.com/apache/zeppelin/pull/1399
[ZEPPELIN-1182] Zeppelin should have Startup and Shutdown message
### What is this PR for?
PR is for logging configuration details of Zeppelin server.
### What type of PR is it?
We can use the below commands to run the zeppelin server in IDE.
mvn antrun:run@start-zeppelin
mvn antrun:run@stop-zeppelin
Thanks,
Kavin
--
View this message in context:
http://apache-zeppelin-dev-mailing-list.75694.x6.nabble.com/How-to-start-Zeppelin-By-IDE-tp14910p14959.html
Sent from the
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/1393
@kavinkumarks thanks for improvement and it makes sense indeed to assign
owner when creating note. just need to address above minor comments as well as
restart CI, some profiles are
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1365
---
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 is
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1398
\cc @Leemoonsoo @AhyoungRyu
---
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 zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/1398
[MINOR] Doc fix for spark interpreter
### What is this PR for?
Several changes on doc of spark interpreter.
* %spark, %sql, %pyspark only works when spark is the default interpreter
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 cloverhearts opened a pull request:
https://github.com/apache/zeppelin/pull/1397
[ZEPPELIN-1406] modify to load method artifact local path in Helium
application json
### What is this PR for?
Currently contained in the Zeppelin Helium application does not work.
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1396
\cc @jongyoul Please review this PR if i misunderstand.
---
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 astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1396
[ZEPPELIN-1405] Remove propertyKeyUnusedConnectionListMap variable in
JDBCInterpreter
### What is this PR for?
This PR is for refactoring code for JDBCInterpreter.
### What
Shim HyungSung created ZEPPELIN-1405:
Summary: Refactoring of JDBCInterpreter.
Key: ZEPPELIN-1405
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1405
Project: Zeppelin
Issue Type:
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/1330
@mwkang thanks for the contribution! actually this is related to multi-user
note management and there's some work going on under #1392 or #1390 .
In your case, i tested it, and
Github user corneadoug commented on the issue:
https://github.com/apache/zeppelin/pull/1388
@minahlee Making a new line is a good improvement because some interpreters
do not handle same line typing.
For removing the default interpreter, I don't have a strong opinion.
If
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1330
@mwkang could you please try close and then re-open this PR to trigger the
CI and see if the same issue persists? Thanks!
\cc @khalidhuseynov for review.
---
If your project is set up for
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/1390
since this also related to #1265 as well as
[ZEPPELIN-1338](https://issues.apache.org/jira/browse/ZEPPELIN-1338) for
interpreter multi user environment, maybe @jongyoul could take more
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1388
@corneadoug actually I made it empty intentionally. please check out the
discussion [here]
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/932
Thank you for addressing licensing issues!
Let me look into that again and then merge to master, if there is no
further discussion.
---
If your project is set up for it, you can reply to
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1248
I'm sorry for delay, @mwkang - my plate was full last week.
---
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 cloverhearts opened a pull request:
https://github.com/apache/zeppelin/pull/1395
[ZEPPELIN-1404] invalid html structure for bootstrap in interpreter setting
page
### What is this PR for?
invalid html structure for bootstrap in interpreter setting page.
for
CloverHearts created ZEPPELIN-1404:
--
Summary: invalid html structure for bootstrap in interpreter
setting page
Key: ZEPPELIN-1404
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1404
Project:
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 zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1265
Thanks @jongyoul , change it to `isolated` works.
Regarding the option complexity, I still have concern on that. So now
there's 2 dimensions for interpreter (user/note) IIUC. Before 0.6,
Github user corneadoug commented on the issue:
https://github.com/apache/zeppelin/pull/1265
@jongyoul Can you make a quick recap on the difference between all of those
options?
(perUser, perNote, shared, scopes, isolated)
---
If your project is set up for it, you can reply to
Github user corneadoug commented on the issue:
https://github.com/apache/zeppelin/pull/1302
My only feedback is that right now, it's hard to know if the interpreter
has finished restarting or not.
My guess is that it takes some time, so you shouldn't be able to restart it
again
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1265
About the first comment you should set `scoped` or `isolated`, not
`shared`. And growth of complexity is to remain backward compatibility. I
also agree that interpreter setting become too
49 matches
Mail list logo