GitHub user minahlee opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/963
[gh-pages][ZEPPELIN-952] Refine website style
### What is this PR for?
Apply same style in website main page as #962
### What type of PR is it?
Documentation
Github user minahlee commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/963
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
Github user minahlee commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/962
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
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1001
Duplicated #992
---
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 minahlee 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
Github user minahlee closed the pull request at:
https://github.com/apache/zeppelin/pull/997
---
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 reopened a pull request:
https://github.com/apache/zeppelin/pull/994
[WIP][ZEPPELIN-986] Create publish release script
### What is this PR for?
This PR is to automate release publish to maven repository.
We used to use maven-deploy-plugin and maven
GitHub user minahlee reopened a pull request:
https://github.com/apache/zeppelin/pull/997
[MINOR] Make nav zeppelin version to point ZEPPELIN_VERSION in _config.yml
### What is this PR for?
One less line to change when releasing docs by making `ZEPPELIN_VERSION` to
be retrieved
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/992
Thanks for catching it. 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 minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/983
Merging it as hotfix
---
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 minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/983
I think it's not changed only in github ui.
you can see that you can access to https://git.apache.org/zeppelin.git/ but
not to https://git.apache.org/incubator-zeppelin.git/
---
If your
GitHub user minahlee opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/959
[ZEPPELIN-951] Update cron expression link and fix broken cron container
style
### What is this PR for?
Update cron expression link and fix broken cron container style
Github user minahlee commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/959
Same test failure is already reported by @bzz in
https://issues.apache.org/jira/browse/ZEPPELIN-942
---
If your project is set up for it, you can reply to this email and have your
Github user minahlee commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/922#issuecomment-92198
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
Github user minahlee commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/921#issuecomment-98108
This is what I get when I direct to Job menu, seems like it doesn't have
valid information though.
```
INFO [2016-05-28 01:51:45,603] ({main
GitHub user minahlee opened a pull request:
https://github.com/apache/zeppelin/pull/1003
[DOCS] Fix link to specific doc from link page
### What is this PR for?
Link to docs page leads to
http://zeppelin.apache.org/releases/docs/0.5.6-incubating instead of
http
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/997
Merging 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 minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1008
@corneadoug I just changed, please check and let me know your opinion.
---
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 minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1011
Merging it into 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
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/981
LGTM, could you rebase? I merged #1011 without knowing that you took care
of hive license here
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1004
https://cloud.githubusercontent.com/assets/8503346/16052985/ad3fa0ec-321b-11e6-9f15-e9725cee700f.png;>
This is what I see from my screen, and cannot scroll down lower. Can you
adj
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1014
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 minahlee opened a pull request:
https://github.com/apache/zeppelin/pull/1013
[ZEPPELIN-1002] Move configuration menus under dropdown
### What is this PR for?
- Move configuration menus under dropdown menu
- Change dropdown menu style
- Change `Login` button
GitHub user minahlee opened a pull request:
https://github.com/apache/zeppelin/pull/1016
Bump up version to 0.7.0-SNAPSHOT
### What is this PR for?
Bump up version to 0.7.0-SNAPSHOT
You can merge this pull request into a Git repository by running:
$ git pull https
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/994
@echarles I think my description caused confusion. Spark will be included
in release since most of Zeppelin user is based on Spark. The modules I wrote
in description is skipped modules
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/994
@echarles
- This script is written exactly because of the reason that @bzz explained
above.
> calling it multiple times with different arguments, instead of single
script was qu
GitHub user minahlee opened a pull request:
https://github.com/apache/zeppelin/pull/1006
[MINOR] Small modification in notebook actionbar
### What is this PR for?
Small ui improvements
- Change keyboard shortcut icon
- Modify commit container style
### What
GitHub user minahlee opened a pull request:
https://github.com/apache/zeppelin/pull/1008
[ZEPPELIN-997] Export data to csv
### What is this PR for?
Add csv download from front-end leveraging #714
### What type of PR is it?
Improvement
### What is the Jira
GitHub user minahlee opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/966
[ZEPPELIN-958] Support syntax highlight for python and r interpreter
### What is this PR for?
Support syntax highlight for python and r interpreter
### What type of PR
Github user minahlee commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/941
@AhyoungRyu Thanks for review I just handled them in bcb6b54
---
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 minahlee opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/941
Remove incubating from doc
### What is this PR for?
Remove incubating from doc
### What type of PR is it?
Documentation
### Questions:
* Does the licenses
Github user minahlee commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/948
Sorry I was confused, let's keep it in `gh-pages`
---
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 minahlee commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/948
We don't update our website with `gh-pages` but with `master` so it will be
still accessible via http://zeppelin.incubator.apache.org/doap.rdf
Thanks for pointing out svn update, I
Github user minahlee closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/948
---
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 minahlee commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/952
Merging it 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 well. If your project does not have this feature
enabled
Github user minahlee commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/912#issuecomment-221996644
@astroshim Did you remove tutorial notebook on purpose? If not, can you
revert the last commit?
---
If your project is set up for it, you can reply
Github user minahlee commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/938
Merging 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
Github user minahlee commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/941
@prabhjyotsingh thanks for pointing it out. I removed doap.rdf file from
doc since it already exists in
https://github.com/apache/incubator-zeppelin/blob/gh-pages/doap.rdf
---
If your
Github user minahlee commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/941
I updated road.pdf file in master branch and removed it from gh-page branch
in #948 for better search and maintenance.
---
If your project is set up for it, you can reply to this email
GitHub user minahlee opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/942
Remove incubating from pom files
### What is this PR for?
Remove incubating from pom files
### Questions:
* Does the licenses files need update
Github user minahlee commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/921#issuecomment-222403825
@cloverhearts Seems like big number of notebooks/paragraphs slows down
loading job manage page. Any idea to solve this issue?
---
If your project is set
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/999
I just tried #1004 and it scroll to expected position. Since #1004 is using
a lot of anchor, would you mind to close this pr so we can reduce burden of
changing every anchor that is used in #1004
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 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
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?
https://cloud.githubusercontent.com/assets/8503346
GitHub user minahlee opened a pull request:
https://github.com/apache/zeppelin/pull/1063
[ZEPPELIN-1045] Apply new mechanism to PythonInterpreter
### What is this PR for?
This PR is applying new interpreter register mechanism to python
interpreter.
### What type of PR
GitHub user minahlee opened a pull request:
https://github.com/apache/zeppelin/pull/1062
[ZEPPELIN-1007] Toggle only one of the interpreter/permission settings
### What is this PR for?
Prevent Interpreter binding and Note Permissions dialog opens at the same
time
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1063
\cc @bzz @jongyoul 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
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 minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1006
I made one more change in table ascending, descending icon. 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
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1006
Selenium test passes after rebasing 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
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1013
Updated selenium test and CI passes
---
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 minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1058
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 minahlee opened a pull request:
https://github.com/apache/zeppelin/pull/1061
[DOC] Fix broken code block in jdbc document
### What is this PR for?
Fix broken code block in jdbc document
### What type of PR is it?
Documentation
### Screenshots
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/458
Duplicated with #933
---
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 minahlee closed the pull request at:
https://github.com/apache/zeppelin/pull/458
---
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 opened a pull request:
https://github.com/apache/zeppelin/pull/1043
[ZEPPELIN-905] Fix import notebook with error result
### What is this PR for?
This PR is fixing import/clone notebook with error result. This PR adds
test based on #933.
### What
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/994
@bzz Thanks for the review. I removed mistakenly pushed file.
I included geode, scalding interpreter in maven publish script so that
geode, scalding users can take advantage of #1042
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
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 minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1070
Thank you for taking a look. Merging it as hot fix
---
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 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 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 minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/994
@Leemoonsoo Thanks for pointing it out! I reverted commit of adding geode,
scalding
---
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 minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1013
I am ok with either approach, seems like @corneadoug has strong opinion on
having two separate menu so I will leave it as it is. I will need to fix
selenium test to make it to be merged though
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1053
Merging it as hotfix
---
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 minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1048
Merging it as hotfix
---
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 minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1013
Thanks for the feedback guys! I've just pushed some commits and updated
description with new screenshot.
---
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/994
@bzz Sorry for late response, I am working on it and going to push commits
by today. Thanks for heads up :)
---
If your project is set up for it, you can reply to this email and have your
reply
GitHub user minahlee opened a pull request:
https://github.com/apache/zeppelin/pull/1020
[HOTFIX][DOC] Fix image path
### What is this PR for?
Use relative path instead of absolute path so doc can find image under
proper version folder
### What type of PR
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/994
@echarles If there are such needs, nothing stops us from publishing it. I
will make change accordingly. Thank you for sharing your situation.
---
If your project is set up for it, you can reply
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1016
@Leemoonsoo Thanks for the explanation!
@felixcheung as @Leemoonsoo said, branch-0.6 will be bumped up to
0.6.1-SNAPSHOT once I create v0.6.0-rc1 tag. We have documented this versioning
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1074
Closes since it is duplicated with #1073
---
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 minahlee opened a pull request:
https://github.com/apache/zeppelin/pull/1074
[HOTFIX] Fix python test case
### What is this PR for?
Fix python test case caused by #1067
### What type of PR is it?
Hot Fix
### How should this be tested?
run
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1075
\cc @bzz 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 wishes
Github user minahlee closed the pull request at:
https://github.com/apache/zeppelin/pull/1074
---
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/1013
Thanks for the feedback all!
@Leemoonsoo After your explanation, it makes sense to display `anonymous`
@khalidhuseynov I added `WebSocket` because logged user can misunderstand
that he
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1066
Merge if there is no more dicussion
---
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 minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/971
LGTM merging it into master and branch-0.6
---
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 minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1071
@prabhjyotsingh Thanks for the fix!
Seems like this PR is bringing CI error in master, branch-0.6:
```
Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 81.986 sec
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1037
Nice catch! Can we make make space to be consistent between `create / edit
/ remove` and `bind/unbind`?
---
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/1043
@swkimme @Leemoonsoo Ready 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
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1033
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 minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1006
Rebased master and updated screenshot in description
---
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 minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1042
Thanks for the effort @Leemoonsoo, @AhyoungRyu I've tested this branch and
it works like a charm! I left few more comments, once they are addressed it's
good to merge :)
---
If your project
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1042
Merging 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 minahlee opened a pull request:
https://github.com/apache/zeppelin/pull/1075
[HOTFXI] Fix python test case
### What is this PR for?
Update `testPy4jIsNotInstalled `, `testPy4jIsInstalled` test
- `z.show` -> `def show` to check `show` function is defi
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1031
It looks great to me. Thanks for improving document
---
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 minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1098
Merging it to master and branch-0.6 so I can create 0.6.0-rc1
---
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 minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1062
Merging 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 minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1040
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 minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1039
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 minahlee opened a pull request:
https://github.com/apache/zeppelin/pull/1089
[DOC] Update doc image in Explore Zeppelin UI page
### What is this PR for?
There were small changes in Zeppelin UI after #1006, #1013, #1081 so update
screenshot of documents accordingly
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1084
Awesome!!! let's merge 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 feature
enabled
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1059
Merge it 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 minahlee opened a pull request:
https://github.com/apache/zeppelin/pull/1098
[HOTFIX] update release script
### What is this PR for?
- Add sparkr, r profile to netinst release build
- Exit release_publish.sh in case creating staging repository fails
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1013
I also don't feel the necessity of showing full user name since logged user
would know his own username. Let me remove username under dropdown then.
---
If your project is set up for it, you
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1013
@corneadoug Having separate menu for the anonymous user totally makes sense.
Then the next question would be where do we want to place it?
Next to Notebook? - will give less confusion who
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1022
@corneadoug yes, the doc for 0.6.0 will be published based on branch-0.6
---
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 minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1013
One question, if `Confidential` is per user menu, should it be shown when
only user logged in?
---
If your project is set up for it, you can reply to this email and have your
reply appear
1 - 100 of 353 matches
Mail list logo