Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1055
@sixmen Yeah thanks for clarifying it.
---
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 minahlee opened a pull request:
https://github.com/apache/zeppelin/pull/1070
[HOTFIX] Fix compile error caused by merging #1043
### What is this PR for?
Hotfix for compile error of master, branch-0.6 after merging #1043
You can merge this pull request into a Git
Github user sixmen commented on the issue:
https://github.com/apache/zeppelin/pull/1055
It works well in my test.
---
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 minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1042
@Leemoonsoo Could you rebase and resolve conflicts?
---
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 Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1058
@astroshim No worries. Thanks for the quick response.
LGTM and merge into master branch if there're no more discussions.
---
If your project is set up for it, you can reply to this email
Github user corneadoug commented on the issue:
https://github.com/apache/zeppelin/pull/1059
Tested, No more handsontable duplicates.
Was there no other way to just update the data instead of destroying the
table and creating a new one?
---
If your project is set up for it, you
Github user corneadoug commented on the issue:
https://github.com/apache/zeppelin/pull/1066
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 and wishes so, or if the
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/994
---
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 corneadoug commented on the issue:
https://github.com/apache/zeppelin/pull/1013
I agree with @khalidhuseynov that having Connected/Disconnected being
replace by the username isn't great.
However I still think that showing a username cf: 'anonymous' for an app
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1013
@khalidhuseynov Right. I totally agree with you.
@minahlee It's more intuitive than before ð
---
If your project is set up for it, you can reply to this email and have your
reply
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1013
@khalidhuseynov Thanks for the feedback.
I totally agree that showing `Connected` message for non-logged user can be
confusing.
I made several changes based on your opinion.
- show
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1063
Looks awesome to me, thank you!
I think it's ready to merge, if there is no further discussion
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1067
Thank you guys for the prompt reviews!
@Leemoonsoo agree on API change and I'm glad to have it merged in 0.6,
updated JIRA
@minahlee review addressed in 3b1ad36
If there is no
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/1013
I've just went through the conversation here, and i think that having
`userName` instead of `connected` status in authenticated mode with additional
`logout` menu is quite misleading.
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1067
I left few notes on documentation and `help() ` function. 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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1058
@Leemoonsoo Sorry for late response. I fixed.
---
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
Ahyoung created ZEPPELIN-1050:
-
Summary: Cannot scroll down if there are many notebooks
Key: ZEPPELIN-1050
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1050
Project: Zeppelin
Issue Type:
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1042
It's ready to review and merge.
Tried updated create_release.sh script. `zeppelin-0.6.0-bin-all.tgz` is
about 498M while `zeppelin-0.6.0-netinst.tgz` will be about 208MB.
---
If your
GitHub user fvaleri opened a pull request:
https://github.com/apache/zeppelin/pull/1069
[ZEPPELIN-281] Windows7 build2
### What is this PR for?
For next release I would like to do simple additions to my proxy setup on
main README:
- Add Maven proxy configuration example.
Github user ggdupont commented on the issue:
https://github.com/apache/zeppelin/pull/908
No luck, it can't get to load my jar from my local repo (nexus on docker).
My request :
{
"artifact": "org.apache.zeppelin:zeppelin-neo4j:0.6.0-SNAPSHOT",
"className":
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1055
@corneadoug Thank you for sharing your opinion and I agree with you. So I
made it to support below two date format for backward compatibility.
- `-MM-dd'T'HH:mm:ssZ` : exported after
Github user cloverhearts closed the pull request at:
https://github.com/apache/zeppelin/pull/1056
---
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
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/1056
pass
more retry ci test.
---
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 cloverhearts reopened a pull request:
https://github.com/apache/zeppelin/pull/1056
[TEST] Selenium Tests 01
only test - selenium
merge - https://github.com/apache/zeppelin/pull/1048 ,
https://github.com/apache/zeppelin/pull/1053
Please, DO NOT
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1058
@astroshim Now there're commits not belongs to this contributions, 41a7302,
fd715c8, ab5ce62
Could you remove these commits?
---
If your project is set up for it, you can reply to this
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1058
The selenium test has passed!!
---
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 Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1067
I saw https://issues.apache.org/jira/browse/ZEPPELIN-1048 marked 0.7.0 as a
fix version.
Not a strong opinion, and i don't think it should be blocker for 0.6
release. But If we're
GitHub user cloverhearts opened a pull request:
https://github.com/apache/zeppelin/pull/1068
[TEST] CI -BUILD TEST - TASK 1
TEST PR
![cap 2016-06-23
02-21-46-486](https://cloud.githubusercontent.com/assets/10525473/16276424/46c26e56-38e9-11e6-945e-45d8a361543e.png)
You
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/1057
Test done.
Closed.
---
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 cloverhearts closed the pull request at:
https://github.com/apache/zeppelin/pull/1057
---
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
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1067
It is ready to be merged now.
TODOs will be addressed in different PR, after #1063 is merged
---
If your project is set up for it, you can reply to this email and have your
reply appear on
GitHub user bzz opened a pull request:
https://github.com/apache/zeppelin/pull/1067
ZEPPELIN-1048: Pandas support for python interpreter
### What is this PR for?
### What type of PR is it?
Feature
### Todos
* [x] fix NPE in logs on empty paragraph
Github user JHowix commented on the issue:
https://github.com/apache/zeppelin/pull/795
Done, thank you for the quick response!
https://issues.apache.org/jira/browse/ZEPPELIN-1049
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
John J. Howard created ZEPPELIN-1049:
Summary: Stop Interpretters at Notebook level when Running in
Isolated and Scoped Mode
Key: ZEPPELIN-1049
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1049
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1058
@Leemoonsoo I rebased.
---
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
Alexander Bezzubov created ZEPPELIN-1048:
Summary: Python: pandas support using Table display system
Key: ZEPPELIN-1048
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1048
Project:
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/994
Merge if there is no more discussion
---
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 Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/795
@JHowix It does not include functionality to restart interpreter at the
notebook level.
I think it really make sense restart interpreter at the notebook level in
case of 'scoped' or
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1059
\cc @corneadoug for 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 have this feature
enabled and
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1043
Merging it into master and branch-0.6 if there is no more discussion
---
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 suvam97 commented on the issue:
https://github.com/apache/zeppelin/pull/1066
@corneadoug : sorry for late response ! Changes updated.
---
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 Leemoonsoo closed the pull request at:
https://github.com/apache/zeppelin/pull/1065
---
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 Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1013
CI failure is irrelevant to this change.
Tested and it looks good to me. +1 for merge it into master and branch-0.6
---
If your project is set up for it, you can reply to this email and
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1065
Thanks @bzz @minahlee taking a look.
i'm merging it into branch-0.6 as a hotfix.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1064
@bzz I fixed as your comments. 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 does not have this
Github user corneadoug commented on the issue:
https://github.com/apache/zeppelin/pull/1066
@suvam97 Sorry, I was actually testing your branch :)
Works, great, could you then delete the `notebookFilter` since it is not
being used?
---
If your project is set up for it, you can
Github user corneadoug commented on the issue:
https://github.com/apache/zeppelin/pull/1066
@suvam97 Maybe you should try building from master and cleaning your cache.
Current filtering is working perfectly for me
GitHub user suvam97 opened a pull request:
https://github.com/apache/zeppelin/pull/1066
[ZEPPELIN-1047] Add filter to handle upper case
### What is this PR for?
The notebook filter in navbar does not filter the results of note if note
name is given in upper case .
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1063
BTW while we are here, how do you think shall we also set
`zeppelin.python.maxResult` to 1000?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Suvam Mohanty created ZEPPELIN-1047:
---
Summary: Notebook filter in navbar matching is case senstive.
Key: ZEPPELIN-1047
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1047
Project: Zeppelin
Github user corneadoug commented on the issue:
https://github.com/apache/zeppelin/pull/1013
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 and wishes so, or if the
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1063
Looks great to me, thank you @minahlee !
Will merge, if there is no more discussion
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user sagarkulkarni3592 commented on the issue:
https://github.com/apache/zeppelin/pull/971
Also, observed this. When the notebook name is shorter, buttons appear far
from the name or box as shown.
![screen shot 2016-06-22 at 12 48 00
Github user sagarkulkarni3592 commented on the issue:
https://github.com/apache/zeppelin/pull/971
@minahlee Done the changes necessary to make the overlap go away.
![screen shot 2016-06-22 at 12 45 34
Github user corneadoug commented on the issue:
https://github.com/apache/zeppelin/pull/1050
Tested, 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 and wishes so,
GitHub user Leemoonsoo opened a pull request:
https://github.com/apache/zeppelin/pull/1065
FIX merge mistake on Branch 0.6
### What is this PR for?
There were some mistake merging https://github.com/apache/zeppelin/pull/998
into branch-0.6
### What type of PR is it?
Github user Leemoonsoo closed the pull request at:
https://github.com/apache/zeppelin/pull/1046
---
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 minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/971
Thanks for the update. I see one more thing needs to be handled.
Could you make btn group not to cover input box in below screenshot?
58 matches
Mail list logo