Github user yoonjs2 commented on the issue:
https://github.com/apache/zeppelin/pull/1294
@AhyoungRyu Thanks for comment! Of course I 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 your project does not have thi
Github user mfelgamal commented on the issue:
https://github.com/apache/zeppelin/pull/1255
@zjffdu do you have any further comments ?
---
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 feat
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1294
@yoonjs2 Thank you for taking care of this. Could you update the below
sentence as well in [this
section](https://github.com/yoonjs2/zeppelin/blob/5ac7de7a4f235cd7b1f27cc4d3c5c76d35dca019/docs/
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/1293
@Leemoonsoo thanks for review. i addressed your comments by changing all
the invalid cases to log in debug mode instead of throwing exception. let me
know if there's something else left.
Github user corneadoug commented on the issue:
https://github.com/apache/zeppelin/pull/1229
@astroshim Was there a JIRA issue?
---
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
ena
Github user corneadoug commented on the issue:
https://github.com/apache/zeppelin/pull/1229
Merged in Master and 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 project does not have this feature
enable
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1229
---
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 e
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1280
LGTM. Merging it into master if there're no further discussions.
---
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 Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1270
@prasadwagle Thanks for the fix!
LGTM. I don't think it's blocker for 0.6.1 release, but it will be nice if
we can include it in 0.6.1 release.
---
If your project is set up for it, you ca
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1290
Thanks @mwkang for the contribution. Tested and it works well.
However, i'm not sure about passing informations like "isKeyEvent",
"moveFocusToNextParagraph", etc, through `paragraph.co
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1298
---
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 e
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1298
Merge into master 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 Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1278
Thanks for the contribution.
Tested and it works great.
Could you rebase and see if it passes CI?
---
If your project is set up for it, you can reply to this email and have your
r
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1279
Thanks for the contribution.
Merge if there're no more discussions.
---
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 p
GitHub user Leemoonsoo opened a pull request:
https://github.com/apache/zeppelin/pull/1298
[HOTFIX] After hotfix #1292, build fails with -Pyarn
### What is this PR for?
#1292 fixed CI problem. However, build command fails with -Pyarn
```
[ERROR] 'dependencies.depend
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1293
Thanks @khalidhuseynov for the fix. I tried this patch and it does not
print stacktrace on PING message anymore when user not logged in.
However, it still prints stacktrace when user br
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1291
CI is green.
Merge to master and branch-0.6 if there're no more discussions.
---
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 Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1269
@zjffdu master branch now fixes the CI problem. Could you rebase master and
see how CI test goes?
---
If your project is set up for it, you can reply to this email and have your
reply appear o
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1257
@prabhjyotsingh I just updated screen-shot. Thank you for taking care of
this.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. I
Github user beriaanirudh commented on the issue:
https://github.com/apache/zeppelin/pull/1283
@jongyoul: I have incorported all feedback:
1. Using ScheduledExecutorService for ensuring AppendOutputRunner is up,
instead of doing so inside the while loop.
2. Synchronizing on the
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1289
---
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 e
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1287
---
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 e
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1292
issue created https://issues.apache.org/jira/browse/ZEPPELIN-1303
---
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
Lee moon soo created ZEPPELIN-1303:
--
Summary: Duplicated profiles in spark/pom.xml and
spark-dependencies.xml
Key: ZEPPELIN-1303
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1303
Project: Zepp
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1292
---
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 e
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1292
CI is green now. I'm merging it into master as a hotfix. And creating a
issue to address duplicated profiles in spark/pom.xml and
spark-dependencies/pom.xml
---
If your project is set up for
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1257
I'll try it out, in the mean while, can you update screen-shot in
description.
---
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 WeichenXu123 opened a pull request:
https://github.com/apache/zeppelin/pull/1297
[GIT] add temp directories generated by zeppelin-Rinterpreter
### What is this PR for?
When compile and test zeppelin-Rinterpreter, it will generate temp
directories `/R` and `/tmp`
GitHub user WeichenXu123 opened a pull request:
https://github.com/apache/zeppelin/pull/1296
[ZEPPELIN-1302] fix rinterpreter default prop values init error
### What is this PR for?
1.
Correct the calling to `SparkInterpreter.getSystemDefault`
from
`SparkInterpreter.g
Weichen Xu created ZEPPELIN-1302:
Summary: RInterpreter props initial error, when calling
getSystemDefault
Key: ZEPPELIN-1302
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1302
Project: Zeppelin
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1257
\cc @AhyoungRyu @prabhjyotsingh @bzz @felizbear
I fixed UI like #1236. Please review.
Thank you!
---
If your project is set up for it, you can reply to this email and have your
reply
GitHub user WeichenXu123 opened a pull request:
https://github.com/apache/zeppelin/pull/1295
[ZEPPELIN-1301] fix potential encoding problem in RInterpreter processHTML
DataURI conversion
### What is this PR for?
fix potential encoding problem in RInterpreter processHTML DataURI
Weichen Xu created ZEPPELIN-1301:
Summary: RInterpreter processHTML has character conversion risk
and may cause error.
Key: ZEPPELIN-1301
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1301
Proje
GitHub user yoonjs2 opened a pull request:
https://github.com/apache/zeppelin/pull/1294
Change maven version from 3.3.3 to 3.3.9
### What is this PR for?
Change maven version from 3.3.3 to 3.3.9 in vagrant script and
documentation due to path to 3.3.3 doesn't exist (return 404 f
Github user khalidhuseynov closed the pull request at:
https://github.com/apache/zeppelin/pull/1293
---
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 feat
GitHub user khalidhuseynov reopened a pull request:
https://github.com/apache/zeppelin/pull/1293
[ZEPPELIN- 1298] Log instead of throwing trace for ping messages
### What is this PR for?
When non authenticated user in non-anonymous mode tries to send any
websocket api message it
GitHub user khalidhuseynov opened a pull request:
https://github.com/apache/zeppelin/pull/1293
[ZEPPELIN- 1298] Log instead of throwing trace for ping messages
### What is this PR for?
When non authenticated user in non-anonymous mode tries to send any
websocket api message it r
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1290
@mwkang Thanks for the improvement! Tested locally and it works well as
expected.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as wel
38 matches
Mail list logo
