GitHub user minahlee opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/913
[MINOR] switch location of interpreter search input box
### What is this PR for?
Make search interpreter input box always to be placed upper than repository
information or
Github user minahlee commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/912#issuecomment-221435706
@astroshim Seems like folder structure notebook is broken. Could you take a
look?
Attaching screenshot
**Master**
https
Github user minahlee commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/899#issuecomment-221436945
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
Github user minahlee commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/912#issuecomment-221786079
@astroshim Thanks for the contribution. One feedback from me is that it
would be awesome if we can filter subdirectory names in nav bar like main page
does
Github user minahlee commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/912#issuecomment-221789099
@astroshim sounds good :)
---
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 minahlee commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/913#issuecomment-221831870
@bzz Rebase is done, the failure seems like related to network issue for
downloading spark.
---
If your project is set up for it, you can reply to this
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 to
Github user minahlee commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/921#issuecomment-86727
I cannot see any job when I visit job tab
https://cloud.githubusercontent.com/assets/8503346/15624760/0e16e062-2446-11e6-9db3-0357e17e2143.png
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 this
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] (
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 pull request:
https://github.com/apache/incubator-zeppelin/pull/605
@H4ml3t Sorry for the confusion, I also confirmed that this commit is not
included in 0.5.6 binary package by checking source code in
http://www.apache.org/dyn/closer.cgi
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 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. If
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? No
* Is there
Github user minahlee commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/943
@jongyoul changes looks good to me. About the documentation I think same as
@bzz for the users who were already using hive interpreter. I think it would be
good to keep hive.md document
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 this
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 opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/948
Remove doap.rdf from gh-page branch
### What is this PR for?
Keep doap.rdf file only in master branch to reduce redundancy since we
moved gh-page contents to master long time ago
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 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 upda
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, or if the
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
GitHub user minahlee opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/952
[HOTFIX] remove incubating from website nav menu
### What is this PR for?
Remove incubating from website nav menu
### What type of PR is it?
Hot Fix
You can merge this
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 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 issue:
https://github.com/apache/incubator-zeppelin/pull/959
@prabhjyotsingh Test is passing now! Thanks for taking care of 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
Github user minahlee commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/959
LGTM. Will Merge this if 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
GitHub user minahlee opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/962
[ZEPPELIN-952] Refine website style
### What is this PR for?
- update document style (font, line-spacing)
- apply same formats for documents
- fix broke document styles
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 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 is
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 this
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 this
Github user minahlee commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/971
It would be better if we make it to be responsive to window width size.
Attaching my screenshot
https://cloud.githubusercontent.com/assets/8503346/15904174/cf0b50c4-2d64-11e6-90d0
GitHub user minahlee opened a pull request:
https://github.com/apache/zeppelin/pull/983
[ZEPPELIN-980]Move git repository from incubator-zeppelin to zeppelin
### What is this PR for?
git repo infra have moved from incubator-zeppelin to zeppelin
### What type of PR is it
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
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/979
@corneadoug I like the idea, but how do we want to handle paragraph
deletion by keyboard shortcut in this case?
---
If your project is set up for it, you can reply to this email and have your
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/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 and
GitHub user minahlee opened 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-release
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/971
I would prefer to keep `noteAction` size as default height. As a Zeppelin
user I prefer to see more of my work space than notebook name and buttons.
---
If your project is set up for it, you can
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/979
Tested, looks good. How about applying @corneadoug's opinion making the
first paragraph not to show `Move Up`, last paragraph not to show `Move Down`?
---
If your project is set up for it
GitHub user minahlee opened 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/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 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 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 is
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 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 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 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 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 opened a pull request:
https://github.com/apache/zeppelin/pull/1011
Update licenses
### What is this PR for?
Update outdated license version
### What type of PR is it?
License
You can merge this pull request into a Git repository by running
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 and
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 on
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1008
@felixcheung Thanks for coming up with the issue, let me handle it in
another PR. I created [new
issue](https://issues.apache.org/jira/browse/ZEPPELIN-1001) for that.
---
If your project is
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
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/976
@cloverhearts Seems like integration test fails, can you fix it please?
```
Tests run: 8, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 120.219 sec
<<< FAIL
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/994
@bzz Thank you for the review, I handled style in 21ca610. Please let me
know if you have any other suggestions :)
---
If your project is set up for it, you can reply to this email and have your
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/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 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 commented on the issue:
https://github.com/apache/zeppelin/pull/1004
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/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, or if the
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 for
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1016
@Leemoonsoo it's already there
---
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
en
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1016
no problem!
---
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
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 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/1013
I think it is common practice to have settings under profile dropdown menu.
### Examples
**Github**
https://cloud.githubusercontent.com/assets/8503346/16098300/2999a8e0-3307-11e6
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 is it
GitHub user minahlee opened a pull request:
https://github.com/apache/zeppelin/pull/1022
[HOTFIX][DOC] Fix image path again
### What is this PR for?
Realized that #1020 doesn't do the job. This PR does fix the issue of
broken image on website and already documented o
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1022
Thank you for the tip! wont make same mistake again...
---
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 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
@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 FYI you don't need to push it to svn since I already did.
---
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 minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1022
Ok let me merge it then :)
---
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/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 on
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
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 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/1032
I hope we have more example notebooks. Once we have a lot of example
notebooks, it won't be efficient to keep all of them embedded in Zeppelin by
default. I guess we will manage most of ex
Github user minahlee commented on the issue:
https://github.com/apache/zeppelin/pull/1013
What about showing user name under dropdown only when username length
exceeds `MAX_USERNAME_LENGTH`?
---
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/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
GitHub user minahlee opened a pull request:
https://github.com/apache/zeppelin/pull/1036
[MINOR] Add .Rhistory in .gitignore
### What is this PR for?
Remove .Rhistory from git file tracking
You can merge this pull request into a Git repository by running:
$ git pull https
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 is
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 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/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/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/971
@sagarkulkarni3592 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
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, or if the
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 opened a pull request:
https://github.com/apache/zeppelin/pull/1045
Revert "Alert message when trying to delete last paragraph."
### What is this PR for?
This reverts commit fc72e12 to test if it caused CI issue. If so, the PR
must come with the t
Github user minahlee closed the pull request at:
https://github.com/apache/zeppelin/pull/1045
---
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/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.
To
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
1 - 100 of 453 matches
Mail list logo
