Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2442
Could you create another PR to trigger the build ?
---
Github user mebelousov commented on the issue:
https://github.com/apache/zeppelin/pull/2442
Cache is clean now. I try to retrigger from both accounts. The result is
the sameÑ
---
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2442
ping @tinkoff-dwh @mebelousov
---
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2442
@mebelousov Could you clean the travis cache and for a dummy force push to
trigger the build ?
---
Github user mebelousov commented on the issue:
https://github.com/apache/zeppelin/pull/2442
@zjffdu I have retriggered several times. Travis fails with the same error
on all jobs.
---
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2442
@tinkoff-dwh @mebelousov Could you retrigger the build ? Travis seems fail
---
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2442
@tinkoff-dwh Could you rebase the PR ?
---
Github user Sanikadze commented on the issue:
https://github.com/apache/zeppelin/pull/2442
+1
---
Github user Aspekt112 commented on the issue:
https://github.com/apache/zeppelin/pull/2442
+1
---
Github user anermakov commented on the issue:
https://github.com/apache/zeppelin/pull/2442
+1 Must have
---
Github user kapustor commented on the issue:
https://github.com/apache/zeppelin/pull/2442
+1
Must-have feature for enterprise usage. Pre-code may be a
resource-management code, or a security code, etc. - users must not have
possibility to change it, only admins.
---
Github user mebelousov commented on the issue:
https://github.com/apache/zeppelin/pull/2442
@zjffdu
hooks are user defined code.
We want to have opportunity to log the user sessions, to bind queries to
notes. It can be possible only by force execution of special query before
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2442
Sorry @mebelousov I don't get why the general hooks could not satisfy you.
The hook can run for each paragraph. Could you explain a little bit more ?
Maybe an example would be very helpful.
Github user mebelousov commented on the issue:
https://github.com/apache/zeppelin/pull/2442
@zjffdu
The hooks as pre-code were discussed [previously
](https://github.com/apache/zeppelin/pull/2096#issuecomment-284220634)
We want precode execute before pre-hook for easy finding
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2442
@tinkoff-dwh I just realised that zeppelin already support hooks for
interpreter. I think you don't need to do it specifically for jdbc interpreter,
instead we could introduce one general property
Github user mebelousov commented on the issue:
https://github.com/apache/zeppelin/pull/2442
@zjffdu, you are right, sessionPrecode is renamed to statementPrecode.
Since 0.8.0 I will recommend to run JDBC interpreter per note in isolated
mode due to Interpreter Lifecycle
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2442
Mostly LGTM, one concern is about the name `session` which might confuse
users. How about changing it to `statement` ?
Besides, I am curious to know how do you use jdbc interpreter. Which
Github user mebelousov commented on the issue:
https://github.com/apache/zeppelin/pull/2442
@zjffdu @felixcheung
Every day our users run about 1000 queries in the Greenplum (open source
Postgres-based DB).
There is great need for binding the query to the user and/or to
Github user tinkoff-dwh commented on the issue:
https://github.com/apache/zeppelin/pull/2442
Ready to review
---
Github user tinkoff-dwh commented on the issue:
https://github.com/apache/zeppelin/pull/2442
Ready to review
---
Github user dwhsys commented on the issue:
https://github.com/apache/zeppelin/pull/2442
Can someone review please?
---
Github user tinkoff-dwh commented on the issue:
https://github.com/apache/zeppelin/pull/2442
Ready to 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 tinkoff-dwh commented on the issue:
https://github.com/apache/zeppelin/pull/2442
Ready to 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 tinkoff-dwh commented on the issue:
https://github.com/apache/zeppelin/pull/2442
Ready to 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 tinkoff-dwh commented on the issue:
https://github.com/apache/zeppelin/pull/2442
Ready to 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 tinkoff-dwh commented on the issue:
https://github.com/apache/zeppelin/pull/2442
Ready to 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 tinkoff-dwh commented on the issue:
https://github.com/apache/zeppelin/pull/2442
Ready to 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 tinkoff-dwh commented on the issue:
https://github.com/apache/zeppelin/pull/2442
it is JDBC interpretator, set variables in first paragraph will not work
because we have connection pool and the paragraphs will be executed in
different statemnets. What problem with shared
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2442
I am thinking that every interpreter could have one special interpreter for
initializing interpreter setting and custom enviroment requirement for his
note. And do the initialization at the first
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2442
I am not sure whether this is a good idea to put custom initialization code
in interpreter setting. Because I think one goal of notebook is to allow user
to reproduce it easily without change
Github user tinkoff-dwh commented on the issue:
https://github.com/apache/zeppelin/pull/2442
@zjffdu
yep, per statement.
for exp.
There is jdbc interpreter per user (postgres).
There are many users and each user has scheme to work.
1. set search_path='#{user}'
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2442
@tinkoff-dwh What do you mean `database session` ? From the code, `precode`
is executed per statement. And what is the usage scenario for this ? Could
you give an example ? Thanks
---
If your
Github user tinkoff-dwh commented on the issue:
https://github.com/apache/zeppelin/pull/2442
@felixcheung
session == database session. added documentation
---
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
33 matches
Mail list logo
