GitHub user echarles opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/385
ZEPPELIN-337 Configurable ContextPath
Allow to configure the context path of the webapp via
zeppelin.server.context.path property or ZEPPELIN_CONTEXT_PATH env variable +
add doc on
GitHub user echarles opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/386
ZEPPELIN-383 CLASSPATH_OVERRIDES environment variable
Prepend CLASSPATH_OVERRIDES environment variable when building CLASSPATH in
the shell scripts.
You can merge this pull request
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/385#issuecomment-153723475
From https://s3.amazonaws.com/archive.travis-ci.org/jobs/88620950/log.txt,
the build is success
[INFO] Zeppelin: Packaging distribution
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/386#issuecomment-153724315
@Leemoonsoo I changed the bin/zeppelin.sh script to print the build
CLASSPATH just after the execution of the java command with
```
echo
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/386#issuecomment-153731822
I rebased here, but should have done for #385
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user echarles closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/386
---
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 echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/386#issuecomment-154452108
#398 replaces this one.
---
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 echarles opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/398
ZEPPELIN-383 CLASSPATH_OVERRIDES environment variable (2)
Prepend CLASSPATH_OVERRIDES environment variable when building CLASSPATH in
the shell scripts.
This PR replaces the
GitHub user echarles opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/412
ZEPPELIN-383 Override classpath with ZEPPELIN_CLASSPATH_OVERRIDES
Prepend ZEPPELIN_CLASSPATH_OVERRIDES environment variable when building
CLASSPATH in the shell scripts.
This
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/398#issuecomment-155117201
I deleted by mistake my fork... Creating #412 to further update this PR
with ZEPPELIN_CLASSPATH_OVERRIDES.
---
If your project is set up for it, you can
Github user echarles closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/398
---
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 echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/422#issuecomment-156042944
@cloverhearts thx for the fix. 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
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/385#issuecomment-156348050
Any more feedback 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
Github user echarles closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/385
---
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 echarles opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/429
Zeppelin 337 configurable context path (2)
Allow to configure the context path of the webapp via
zeppelin.server.context.path property or ZEPPELIN_SERVER_CONTEXT_PATH env
variable
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/385#issuecomment-156697112
Please use #429 to track 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
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/429#issuecomment-156697172
Travis fails with
09:59:03,926 INFO org.apache.zeppelin.rest.AbstractTestRestApi:247 -
Connecting to http://localhost:8080/api/
09:59:03,929 INFO
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/429#issuecomment-156730751
@jongyoul just pushed an update and check is now successful.
It is common practice to define the context path in any webapp. Sometimes
you don&#
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/429#issuecomment-156790437
@jongyoul my intent here is not directly linked to serving different user.
I just need to have zeppelin respond to a well defined context path.
---
If
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/429#issuecomment-158003641
@jongyoul @Leemoonsoo Thank you for reviewing this and pointing remaining
issues.
I have fixed the behavior in the distribution package (simply
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/447#issuecomment-158169477
Can we also cover insertion via keyboard shortcuts, ideally the same as
ipython (http://johnlaudun.org/20131228-ipython-notebook-keyboard-shortcuts
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-170337289
I have sent a mail to the dev list
http://mail-archives.apache.org/mod_mbox/incubator-zeppelin-dev/201512.mbox/%3C5683E471.9010001%40apache.org%3E
GitHub user echarles opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/702
R and SparkR Support [WIP]
### What is this PR for?
Implement R and SpakR Intepreter as part of the Spark Interpreter Group. It
also implements R and Scala binding (in both
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-180962488
CLI fails for profiles 1.3, 1.2 and 1.1 with `object r is not a member of
package org.apache.spark.api` which is normal as SparkR was introduced in 1.4
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-181417475
@Leemoonsoo
- rscala is now downloaded at build time
- bin_license updated
- Unit test added
- spark-1.3, spark-1.2, spark-1.1 should now
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-181565303
2 failures on last build:
spark-1.3:
ZeppelinIT.testSparkInterpreterDependencyLoading:229->AbstractZeppelinIT.waitForParagraph
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-181568340
@echarles @justsomeone1001 This PR supports ggplot2 and displaying R
dataframe in native zeppelin widget
## ggplot2
[ - maybe some code snippets?
---
If your project is set up for it, you
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/703#issuecomment-187057575
Or "Separate interpreter for each note" ?
cfr. https://issues.apache.org/jira/browse/ZEPPELIN-616 Zeppelin UI
consistency: should u
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-187717158
@btiernay Just pushed the HTML, TABLE and IMG display support (the dynamic
form is not yet fully operational).
[![R
Display](https
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-187781300
@elbamos just pulled the 208 patch, built and tested with "%r print("%html
hello")". Got
```
"[1] "%html hell
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/754#issuecomment-189898912
I often use the stacktraces to debug what's going wrong. With this, I
expect I will miss information. Can we make this configurable (default being
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/754#issuecomment-190602637
@zhongneu I will rather give you a message a user could face running the
SQL: `java.lang.IllegalArgumentException: requirement failed` What will the guy
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/754#issuecomment-190609552
So we all agree to make it configurable on the interpreter settings leve
`zeppelin.spark.sql.stacktrace = false` by default ?
What about
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/754#issuecomment-193176072
Tested with zeppelin.spark.sql.stacktrace being false and true, and it
works fine.
Good to merge.
PS: With HEAD source, I have issues
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/754#issuecomment-193657371
@AhyoungRyu I have opened ZEPPELIN-725 for the interpreter update issue.
@felixcheung @Leemoonsoo This PR #754 is good to merge.
---
If your
GitHub user echarles opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/768
Interpeter settings update must set the properties
### What is this PR for?
If you change the intrepreter setting via the UI, the updated values are
sent to the REST API but
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/768#issuecomment-193829319
@AhyoungRyu Thx. I am now facing another issue when registering an
additional interpreter (nothing related to this PR, but I wonder if recent #703
could be
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-193870020
@jeffsteinmetz #lol is it really working for you? After the last merge with
master, the r interpreter is no more taken in the settings... I am double
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-193883993
I guess knitr uses evaluate under the hood
(https://cran.rstudio.com/web/packages/evaluate/index.html mentions it as
reverse import and https
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-193902973
Never tried `IRkernel/repr` nor `Base64enc`...
The process would be :
- Install them
- Try to generate a plot / image
- Report result
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-193914591
Same answer for `mplot` as for `IRkernel/repr` or `Base64enc`.
We have to install and try (If you don't have time, I take note of them and
wil
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-194109402
Right, `%r print("title
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-200906016
@Leemoonsoo I have a list of comments and actions (mainly coming from
@jeffsteinmetz) I had planned to work out this weekend. You PR comes to the
point. I
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/795#issuecomment-201143067
I don't in the src any change on the SparkInterpeter to deal with this new
conf.
About default, sharing variables between notes is sometime
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/795#issuecomment-201153577
Also, should non `shared` modes be marked as `experimental` (see
ZEPPELIN-761). There could be unexpected side effects while using the Spark
REPL in non
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/789#issuecomment-201156913
Current patch sets env in RemoteInterpreter and system properties in
RemoteInterpreterServer. Why is this unbalanced (I would have expected setting
env and
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/789#issuecomment-201174476
Thx, Makes sense.
The rolls would be to have separated ui for env variables, but having env
and java properties mixed in the interpreter settings is
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-202013345
I have merged datalayer#7.
(a) Issues with `spark.home` not being taken into account is fixed.
(b) Support of additional viz libs mentioned by
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-202757005
@jeffsteinmetz Thx for the review and inputs.
- I have pushed a fix that shows `Success`(upper right of the paragraph)
when the returned output is
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-203227621
Huge thx @Leemoonsoo for digging into the interpreter list. I should have
made it by myself, but running out of time atm... I have merged and travis
shows
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-203283238
@jeffsteinmetz thx, just pushed a change to show the unlist example.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-204327315
Thx @prabhjyotsingh for the hint. I have merged and there is no more
conflict.
Travis failed for 1.1.1 profile (issue while downloading a jar
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/810#issuecomment-204409082
That's a nice improvement which will remove some failures, but download
failures also occur on simple jars (writing this down while thinking to
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-204641032
Back to green after simple build retrigger.
LGTM for merge.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/751#issuecomment-204641943
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
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/804#issuecomment-204642561
The expected behavior I would expect is the "Before" one. If I ask for a
result, it should be consistent, whatever page I navigate in between.
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-205118755
Back to green.
Good to merge.
---
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 echarles opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/831
[ZEPPELIN-798] Migrate to Jetty version 9
### What is this PR for?
Upgrade from Jetty 8 to Jetty 9 mainly for deadlock bug fix
https://bugs.eclipse.org/bugs/show_bug.cgi?id=389645
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/831#issuecomment-208932745
```
org.apache.zeppelin.rest.SecurityRestApiTest Time elapsed: 180.035 sec
<<< ERROR!
java.lang.UnsupportedClassVersionError:
org/ecli
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/831#issuecomment-208933458
@anthonycorbacho comments crossing... Just realized yes, we need JDK-1.8...
Should I open a thread on user mailinglist?
---
If your project is set up for
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/831#issuecomment-208934010
Or `9.2.2.v20140723` seems to be happy with jdk1.7... WDYT?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/831#issuecomment-208972267
CI is green *but* I have used 9.2.??? roughly 9 months ago and it have
given "CPU leaks": after a few user requests, the CPU suddenly went to a
c
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/831#issuecomment-209221661
@prasadwagle Reliability repoduce deadlocks is not possible IMHO. However,
you can simulate load with tools like jmeter.
---
If your project is set up for
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/831#issuecomment-212781280
Timeout while downloading jars during travis build...
```
[WARNING] Could not transfer metadata
org.apache.apache.resources:apache-jar-resource
GitHub user echarles reopened a pull request:
https://github.com/apache/incubator-zeppelin/pull/831
[ZEPPELIN-798] Migrate to Jetty version 9
### What is this PR for?
Upgrade from Jetty 8 to Jetty 9 mainly for deadlock bug fix
https://bugs.eclipse.org/bugs/show_bug.cgi?id
Github user echarles closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/831
---
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 echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/831#issuecomment-213308657
@felixcheung back to green.
@prasadwagle Had you the chance to test this branch? Did it solve the
deadlock issue?
---
If your project is set up for it
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/867#issuecomment-215690146
Yes, better tha #863.
To make it clear that the filter is applied to both the `api` and `ws`, I
would move `webapp.addFilter(new FilterHolder
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/31#issuecomment-91997703
Quick input: I favor adding a profile for each version, it makes thinks
clear.
I am confused every time I need to build spark with options
like
GitHub user echarles opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/127
getPort should not have curly bracket in its body
This patch simply remove the usage of curly bracket in the body of the
getPort method of the app.js.
This is needed so the
77 matches
Mail list logo
