Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/497
Haven't had the time to look into that further. TBH, I don't think storing
notebooks as text files is a good strategy - not in the environment we use
Zeppelin, anyway. We're likely
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2599
done :)
---
GitHub user herval opened a pull request:
https://github.com/apache/zeppelin/pull/2599
[ZEPPELIN-2952] encrypt credentials.json with AES
### What is this PR for?
Support encrypting passwords using a private key
### What type of PR is it?
Improvement
Herval Freire created ZEPPELIN-2952:
---
Summary: Support encrypting passwords on credentials.json
Key: ZEPPELIN-2952
URL: https://issues.apache.org/jira/browse/ZEPPELIN-2952
Project: Zeppelin
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2440
Hi folks,
Please let me know if I should close this. Tks!
---
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2498
Hi folks,
Please let me know if I should close this. Tks!
---
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2439
Hi folks,
Please let me know if I should close this. Tks!
---
Github user herval closed the pull request at:
https://github.com/apache/zeppelin/pull/2480
---
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2480
@zjffdu what's the benefit of *not* doing it, given it's a simple change?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as wel
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2503
any arguments against following the current way most sites do it (which is
having "remember me" on by default, instead of a checkbox option)?
Showing/hiding the checkbox depe
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2145
https://github.com/herval/zeppelin/tree/encrypt-credentials
---
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 herval commented on the issue:
https://github.com/apache/zeppelin/pull/2503
Hmm It shouldnât work without the Cookie manager. The checkbox will still
show, though, which begs the question if it should be a default? (Otherwise
weâll have to hide the checkbox based on
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2145
I've implemented something slightly similar here, except I'm actually
encrypting the entire file using AES, using a passphrase you provide (which can
live in memory)
it's
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2484
@jongyoul good q. I tried another stats libraries I've seen used around
(airlift), but had a hard time reconciling dependencies, so went with the
smallest footprint one + implementing the se
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/497
@p0wl I started remote git support here:
https://github.com/herval/zeppelin/tree/remote-git-support - it's a WIP, if
someone wants to take over (it would work w/ github or any other git
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2503
That popup is not the siteâs âremember meâ - github actually keeps
you logged in by default (close the browser and reopen, and youâre still
logged, no need to re-authenticate)
You
GitHub user herval opened a pull request:
https://github.com/apache/zeppelin/pull/2503
[ZEPPELIN-2808] remember me support
### What is this PR for?
Add support for "remember me" cookies, so we can activate LDAP without
forcing users to re-login every time
Herval Freire created ZEPPELIN-2808:
---
Summary: "Remember Me" for Login
Key: ZEPPELIN-2808
URL: https://issues.apache.org/jira/browse/ZEPPELIN-2808
Project: Zeppelin
Issue Type: I
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2439
@soralee @khalidhuseynov would you or anyone else like to give this a new
try?
---
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 herval commented on the issue:
https://github.com/apache/zeppelin/pull/2484
Hi folks - any thoughts on this approach?
---
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 herval commented on the issue:
https://github.com/apache/zeppelin/pull/2498
Hi folks,
Any thoughts on this?
---
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 herval opened a pull request:
https://github.com/apache/zeppelin/pull/2498
[ZEPPELIN-2796] Fetching LDAP roles
### What is this PR for?
This actually fetches the user's LDAP roles, when using the LdapRealm, so
they're accessible on Zeppeli
Herval Freire created ZEPPELIN-2796:
---
Summary: LDAP fetching of roles not working?
Key: ZEPPELIN-2796
URL: https://issues.apache.org/jira/browse/ZEPPELIN-2796
Project: Zeppelin
Issue Type
Herval Freire created ZEPPELIN-2792:
---
Summary: Hide UI elements the user doesn't have credentials for
Key: ZEPPELIN-2792
URL: https://issues.apache.org/jira/browse/ZEPPELIN-2792
Project: Zep
Herval Freire created ZEPPELIN-2791:
---
Summary: Support "super users"
Key: ZEPPELIN-2791
URL: https://issues.apache.org/jira/browse/ZEPPELIN-2791
Project: Zeppelin
Issue Type: N
Herval Freire created ZEPPELIN-2790:
---
Summary: Storing Notebooks and Configs on database
Key: ZEPPELIN-2790
URL: https://issues.apache.org/jira/browse/ZEPPELIN-2790
Project: Zeppelin
Issue
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2440
This is now passing all tests - please advise
---
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 herval commented on the issue:
https://github.com/apache/zeppelin/pull/2440
hmm I'm finding it difficult to get all the integration tests to pass.
Inclined to test only for the presence of the button.
---
If your project is set up for it, you can reply to this emai
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2428
ping - please advise
---
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 herval commented on the issue:
https://github.com/apache/zeppelin/pull/2439
hmm I'll see if I can somehow split "report mode" and "view only mode".
I didnt experience the Helium issue, personally
---
If your project is set up for it, yo
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2480
sounds good, Iâll do that then
_
From: Felix Cheung
mailto:notificati...@github.com>>
Sent: Wednesday, July 12, 2017 9:34 AM
Subject: Re: [
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2439
all tests passing now
---
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 herval commented on the issue:
https://github.com/apache/zeppelin/pull/2484
This is the basic set of functionality needed to start monitoring some
common metrics - Comments & observations welcome!
---
If your project is set up for it, you can reply to this email and
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2480
Regarding verification, how do you suggest I go about that? We've been
using this setup internally for some time, but we only use a few of the
interpreters (spark, scalding, jdbc, md) - not
GitHub user herval opened a pull request:
https://github.com/apache/zeppelin/pull/2484
[ZEPPELIN-2711] [WIP] basic metrics for paragraphs & notebook
view/create/run
### What is this PR for?
This exposes JMX metrics for a few operations on Zeppelin. The first step
her
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2481
oh, Indeed. There's some inheritance 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
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2481
I actually don't see any test on `JdbcInterpreterTest` covering that code
path at all - I think the test suite is passing simply because that bit is not
tested, and feel it should be
---
If
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2481
would it be worth adding a test around this? I imagine the tests would fail
with a "class not found", so it would be easy to add one that proves that you
don't get the error if aut
GitHub user herval opened a pull request:
https://github.com/apache/zeppelin/pull/2480
[ZEPPELIN-1249] build all submodules w/ the same scala version
### What is this PR for?
This uses the same Scala version throughout modules, allowing us to use
only Scala 2.11
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2439
Ok, retested now and it's working as expected.
If you need to login (using anonymous mode support), you may need this PR
first: https://github.com/apache/zeppelin/pull/2440
---
If
Herval Freire created ZEPPELIN-2760:
---
Summary: Can't run JDBC interpreters due to class not found
Key: ZEPPELIN-2760
URL: https://issues.apache.org/jira/browse/ZEPPELIN-2760
Project: Zep
Github user herval commented on the pull request:
https://github.com/apache/zeppelin/commit/e1f0a3205eb5aede0b2d80c9d3de59b3f47b699c#commitcomment-23016085
Related ticket: https://issues.apache.org/jira/browse/ZEPPELIN-1326
---
If your project is set up for it, you can reply to this
Github user herval commented on the pull request:
https://github.com/apache/zeppelin/commit/e1f0a3205eb5aede0b2d80c9d3de59b3f47b699c#commitcomment-23016062
This caused a regression:
https://github.com/apache/zeppelin/blame/master/jdbc/src/main/java/org/apache/zeppelin/jdbc
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2428
Rebased to latest master - please advise
---
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 herval commented on the issue:
https://github.com/apache/zeppelin/pull/2439
Hmm Iâll try again.
Btw, did you `mvn clean` after switching branches? I think changes on the
js/html files donât get picked up unless you do that
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2439
@soralee thanks for taking the time :-) there's one more step to your list:
1. login as user1
2. create a note
3. add a paragraph
**4. change the visualization to a
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2439
any thoughts on this?
---
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 herval commented on the issue:
https://github.com/apache/zeppelin/pull/2440
any additional thoughts on this? Is it mergeable? @jongyoul @cfries
---
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 herval commented on the issue:
https://github.com/apache/zeppelin/pull/2428
any additional thoughts on this? Is it mergeable? @khalidhuseynov
@tinkoff-dwh @felixcheung
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2428
I am using the original setting (getMaxResults). I had to set it in two
calls because setFetchSize will determine that there is a ânextâ page, but
if I only do âsetMaxResultsâ, it will not
Herval Freire created ZEPPELIN-2711:
---
Summary: Metrics for service monitoring
Key: ZEPPELIN-2711
URL: https://issues.apache.org/jira/browse/ZEPPELIN-2711
Project: Zeppelin
Issue Type: New
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2428
@tinkoff-dwh it's not duplicate. The bug here is that if you don't do
`statement.setMaxRows`, some drivers (eg postgres) will try to load everything
when you issue a `select
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2428
---
If your project is set up for it, you can reply to this
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2428
hmm.. It actually seems that setFetchSize does not limit the query
execution (so it takes forever & hangs Zeppelin)
Figuring out if there's a way to show results are trunca
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2439
Build is green: https://travis-ci.org/herval/zeppelin/builds/247345573
---
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 herval commented on the issue:
https://github.com/apache/zeppelin/pull/2428
Build's passing: https://travis-ci.org/herval/zeppelin/builds/247193502
Let me know if this is mergeable :-) @tinkoff-dwh
---
If your project is set up for it, you can reply to
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2440
@zjffdu added one, let me know if that covers 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 herval opened a pull request:
https://github.com/apache/zeppelin/pull/2440
[ZEPPELIN-2587] allow logging in if youre anonymous
### What is this PR for?
The login button is not showing (since 0.7.1) if you allow anonymous
access. This shows it again
GitHub user herval opened a pull request:
https://github.com/apache/zeppelin/pull/2439
[ZEPPELIN-2680] allow opening notebook as a reader
### What is this PR for?
We have a few notebooks that have a list of writers & owners, but no
restricted readers list.
When you
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2428
@tinkoff-dwh build #3 is failing in both my master & zeppelin master:
https://travis-ci.org/apache/zeppelin - restarting didn't make it pass:
https://travis-ci.org/apache/zeppelin/jobs/
Herval Freire created ZEPPELIN-2681:
---
Summary: Fix Scala 2.11 compatibility
Key: ZEPPELIN-2681
URL: https://issues.apache.org/jira/browse/ZEPPELIN-2681
Project: Zeppelin
Issue Type
Herval Freire created ZEPPELIN-2680:
---
Summary: Can't open notebook as a reader
Key: ZEPPELIN-2680
URL: https://issues.apache.org/jira/browse/ZEPPELIN-2680
Project: Zeppelin
Issue Type
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2428
It seems apache/zeppelin master (which is what this branch is based on) is
failing the build - should I rebase it to 0.7 instead, or wait for master to be
fixed? Please let me know what should I
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2428
Why +1?
---
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 herval commented on the issue:
https://github.com/apache/zeppelin/pull/2428
I'm also not exactly sure how to write a test for this, as the
JDBCInterpreterTest doesn't use anything in the line of mocks (and the outside
behavior - limiting the number of returned
Github user herval commented on the issue:
https://github.com/apache/zeppelin/pull/2428
@tinkoff-dwh you're right, picked the wrong method - using setFetchSize now
---
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 herval opened a pull request:
https://github.com/apache/zeppelin/pull/2428
[ZEPPELIN-1470] limiting results from jdbc
### What is this PR for?
One thing we tracked down is that if you issue a large query on a very
large table, it will simply try to load all
67 matches
Mail list logo
