GitHub user prabhjyotsingh opened a pull request:
https://github.com/apache/zeppelin/pull/1101
[ZEPPELIN-1072] Saving Interpreter Setting dosen't notify when its finished
### What is this PR for?
When user tries to save Interpreter Setting, it does not notify user when
its done
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1086
Merging 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 have
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1091
@bzz fair point;
- have refactored all AbstractZeppelinIT's sleep calls into
ZeppelinITUtils.
- added another function clickAndWait, which will click and wait for 1s
GitHub user prabhjyotsingh opened a pull request:
https://github.com/apache/zeppelin/pull/1091
[ZEPPELIN-1065] add delay after deleting paragraph.
### What is this PR for?
This is for fixing flaky CI failing test
```
testRemoveButton
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1052
Merging 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 have
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1080
Ready for review
CI fails for #5326.1, Which is unrelated.
```
---
T E S T S
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1052
Have tested it, it works on a Kerberos enabled system.
---
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 prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1060
Going back i.e. `history.back()`wouldn't for if someone has shared a link
(for example) over chat/email on which you don't have access to. Hence, I
would recommend
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1079
Merging 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 have
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1084
@minahlee @bzz @Leemoonsoo After merging this CI is now failing (Build
#5323.1) with
```
---
T E S T S
GitHub user prabhjyotsingh opened a pull request:
https://github.com/apache/zeppelin/pull/1080
[ZEPPELIN-1060] validate user before saving
### What is this PR for?
Zeppelin notebook permissions change has an issue while reconfiguring
permissions by clearing textbox
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1079
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 prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1071
Merging 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 have
GitHub user prabhjyotsingh opened a pull request:
https://github.com/apache/zeppelin/pull/1071
[ZEPPELIN-1052] Application does not logout user when authcBasic is used
### What is this PR for?
This PR is WRT to
[this](http://apache-zeppelin-users-incubating-mailing-list.75479
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1070
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 prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1051
CI fails for
#5104.1
```
Results :
Failed tests:
ZeppelinSparkClusterTest.sparkRTest:104 expected: but
was:
```
#5104.7
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1051
@jongyoul Thank you for the review, have made relevant change.
---
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 prabhjyotsingh opened a pull request:
https://github.com/apache/zeppelin/pull/1051
increase more logs, and return appropriate error to front-end
### What is this PR for?
In the current implementation JDBC interpret only catches SQLException
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1042
One more feature request for this, with downloading an interpreter,
creation/insertion of relevant entry in `conf/interpreter.json`
---
If your project is set up for it, you can reply
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1027
Merging 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 have
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1028
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 have
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1032
Good one, we should have 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
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1036
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 prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1027
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 prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1028
@bzz I was referring to https://github.com/apache/zeppelin/pull/1034,
ZEPPELIN-1009.
```
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 104.524 sec
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/986
I think we should as this is to do with AD auth.
---
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 prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/986
Merging 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 have
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1029
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 prabhjyotsingh opened a pull request:
https://github.com/apache/zeppelin/pull/1028
[Zeppelin-1022] Apply new mechanism to LivyInterpreter
### What is this PR for?
Apply new mechanism to LivyInterpreter
### What type of PR is it?
[Improvement
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/993
@AhyoungRyu is this still a work in progress ?
Is there a way in which we can go away with seeing all of these three munu
i.e. Interpreter setting, Credentials and Configurations
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/986
@rja1 have made some changes, hope
this(188ac170c7a0921044f301ae703bf580a71b2cdf) solves for
"activeDirectoryRealm.principalSuffix isn't honoured"
---
If your project
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/986
Hi @rja1,
I see two issues in your request;
1. Your AD require you to pass @domainName; I understand in some of the
cases, depending on how you have configured AD
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/986
@vinayshukla, Yes, this fix is for both Active Directory and LDAP. Have
tested this with configuring Active Directory running on Windows Server 2008,
and on an open LDAP running
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/986
@Leemoonsoo thank you for taking a look at it. Have added a selenium test
case for the same.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/996
Sure, I think we can merge this 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
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/986
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
enabled
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/996
One more suggestion not sure if that is relevant to this PR should we have
`5` near
https://github.com/apache/zeppelin/blob/master/spark-dependencies/pom.xml#L827
and
https
GitHub user prabhjyotsingh opened a pull request:
https://github.com/apache/zeppelin/pull/992
[ZEPPELIN-980] missing "incubation-" references
### What is this PR for?
This is extension to https://github.com/apache/zeppelin/pull/983, with
missing references of
GitHub user prabhjyotsingh opened a pull request:
https://github.com/apache/zeppelin/pull/986
[Zeppelin 946] [WIP] Permissions not honoring group
### What is this PR for?
Error:
Insufficient privileges to write notebook.
Allowed users or roles: [admin, zeppelinWrite
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/944
Merging 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 have
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/951
Thanks for the explanation. 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 prabhjyotsingh commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/956
@AhyoungRyu have made a fix #4492.7 failed #958. Can you rebase with
master, and re-trigger ?
And yes other is `[INFO] Rat check: Summary of files. Unapproved: 1
unknown
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/959
Hi @minahlee,
This CI failure is fixed with #958. Can you rebase with master and
re-trigger ?
---
If your project is set up for it, you can reply to this email and have your
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/960
This CI failure was fixed with
https://github.com/apache/incubator-zeppelin/pull/958. Can you rebase with
master and re-trigger ?
---
If your project is set up for it, you can
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/958
Tried it 4 different times
- https://travis-ci.org/apache/incubator-zeppelin/builds/135110386
- https://travis-ci.org/apache/incubator-zeppelin/builds/135119542
GitHub user prabhjyotsingh opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/958
[Hot Fix] Fix flaky ZeppelinIT build
### What is this PR for?
Master is failing for ZeppelinIT at
testSparkInterpreterDependencyLoading:243, as the previous line's
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/937
Tested it, works well. Auto complete was one of the missing feature. ð
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
GitHub user prabhjyotsingh opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/951
Move zeppelin version from home page to about section
### What is this PR for?
Move zeppelin version that is shown home page to new about section.
### What type
Github user prabhjyotsingh commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/933#issuecomment-222641325
@swkimme I think this is failing because you are trying to get paragraph
result i.e. `p.getResult()` or `p2.getResult()` without running any
Github user prabhjyotsingh commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/915#issuecomment-221483256
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 prabhjyotsingh commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/901#issuecomment-221169107
Merging 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
Github user prabhjyotsingh commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/907#issuecomment-221167326
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 prabhjyotsingh commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/911#issuecomment-220962020
Or that is also broken for `$locationChangeSuccess`, its not cleaning up.
Can you fix this as well ?
---
If your project is set up for it, you can
Github user prabhjyotsingh commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/911#issuecomment-220959983
If we use `$scope.search = function(searchTerm)` then we can
[this](https://github.com/apache/incubator-zeppelin/blob/master/zeppelin-web/src
801 - 854 of 854 matches
Mail list logo
