Github user monsieurp commented on the issue:
https://github.com/apache/zeppelin/pull/3239
Yes it is! :)
---
Github user monsieurp commented on the issue:
https://github.com/apache/zeppelin/pull/3239
@zffdu: Sorry for not responding earlier. IMHO, deleting `common.max_count`
should be dealt with in a separate PR / bug report since it is completely
unrelated to the feature here. Until then
Github user monsieurp commented on the issue:
https://github.com/apache/zeppelin/pull/3239
@zhongneu: OK, done.
---
Github user monsieurp commented on the issue:
https://github.com/apache/zeppelin/pull/3239
@zjffdu: Hi! Fair enough. What do you want me to do then? Should I remove
the `rowsFetchSize` property?
---
Github user monsieurp commented on the issue:
https://github.com/apache/zeppelin/pull/3239
@zjffdu: Sorry for taking so long to respond. It's been a busy week. I've
rebased my PR as requested.
---
Github user monsieurp commented on the issue:
https://github.com/apache/zeppelin/pull/3239
Is there anything else I can improve?
---
GitHub user monsieurp opened a pull request:
https://github.com/apache/zeppelin/pull/3241
ZEPPELIN-3880: Code refactoring: remove class files using a Java stream
### What is this PR for?
Make use of a Java stream instead of an ugly for-loop to get rid of class
files
Github user monsieurp commented on the issue:
https://github.com/apache/zeppelin/pull/3239
argh! I give up... :(
---
Github user monsieurp commented on the issue:
https://github.com/apache/zeppelin/pull/3239
I don't know why Jenkins failed. Let's try again.
---
Github user monsieurp closed the pull request at:
https://github.com/apache/zeppelin/pull/3239
---
GitHub user monsieurp reopened a pull request:
https://github.com/apache/zeppelin/pull/3239
Zeppelin 3879: create "maxRows" and "rowsFetchSize" values in
interpreter/jdbc/interpreter-setting.json
### What is this PR for?
This PR introduces code to co
Github user monsieurp commented on the issue:
https://github.com/apache/zeppelin/pull/3239
Thanks @zjffdu and @aka-demik for your comments. I've updated the PR
accordingly.
---
GitHub user monsieurp opened a pull request:
https://github.com/apache/zeppelin/pull/3239
Zeppelin 3879: create "maxRows" and "rowsFetchSize" values in
interpreter/jdbc/interpreter-setting.json
### What is this PR for?
This PR introduces code to configure m
Github user monsieurp closed the pull request at:
https://github.com/apache/zeppelin/pull/2799
---
Github user monsieurp commented on the issue:
https://github.com/apache/zeppelin/pull/2799
Hi @zjffdu. Can I do something to get this PR merged?
---
Github user monsieurp commented on the issue:
https://github.com/apache/zeppelin/pull/2799
https://builds.apache.org/job/zeppelin-pull-request/4115/console says:
```
GitHub pull request #2,799 to apache/zeppelin
[EnvInject] - Loading node environment variables.
Building
Github user monsieurp commented on the issue:
https://github.com/apache/zeppelin/pull/2799
Hi @zjffdu. I'm not quite sure your CI workflow works properly. Jenkins
keeps returning a red flag even though this is a clean PR from scratch.
---
GitHub user monsieurp opened a pull request:
https://github.com/apache/zeppelin/pull/2799
Add a bunch of missing annotations to ActiveDirectoryGroupRealm.java.
### What is this PR for?
Add a bunch of missing annotations to ActiveDirectoryGroupRealm.java. Those
annotations are
Github user monsieurp closed the pull request at:
https://github.com/apache/zeppelin/pull/2794
---
Github user monsieurp commented on the issue:
https://github.com/apache/zeppelin/pull/2794
Ok something is up. I'm going to file a new PR.
---
GitHub user monsieurp reopened a pull request:
https://github.com/apache/zeppelin/pull/2794
ZEPPELIN-3225: Add a bunch of missing annotations to
ActiveDirectoryGroupRealm.java.
### What is this PR for?
Add a bunch of missing annotations to ActiveDirectoryGroupRealm.java. Those
Github user monsieurp closed the pull request at:
https://github.com/apache/zeppelin/pull/2794
---
Github user monsieurp commented on the issue:
https://github.com/apache/zeppelin/pull/2794
Alright.
---
GitHub user monsieurp reopened a pull request:
https://github.com/apache/zeppelin/pull/2794
ZEPPELIN-3225: Add a bunch of missing annotations to
ActiveDirectoryGroupRealm.java.
### What is this PR for?
Add a bunch of missing annotations to ActiveDirectoryGroupRealm.java. Those
Github user monsieurp closed the pull request at:
https://github.com/apache/zeppelin/pull/2794
---
Github user monsieurp commented on the issue:
https://github.com/apache/zeppelin/pull/2794
Hi @zjffdu! Cool. I will file another PR after this one.
I made a mistake, amended my commit and forced pushed it, which is why the
CI build failed. How can I get Jenkins to start a
Github user monsieurp commented on the issue:
https://github.com/apache/zeppelin/pull/2794
Hey @mebelousov! We also use Active Directory at work. We don't go through
the AD connector though but instead rely on the LDAP connector.
By the way, there are many improvements
GitHub user monsieurp opened a pull request:
https://github.com/apache/zeppelin/pull/2794
Add a bunch of missing annotations to ActiveDirectoryGroupRealm.java.
### What is this PR for?
Add a bunch of missing annotations to ActiveDirectoryGroupRealm.java. Those
annotations are
28 matches
Mail list logo
