[GitHub] zeppelin issue #1348: [ZEPPELIN-1313] NullPointerException when using Clone ...

Github user kavinkumarks commented on the issue: https://github.com/apache/zeppelin/pull/1348 @felixcheung / @jongyoul could you please take care of merging this? Thanks, Kavin --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin issue #1241: [ZEPPELIN-1217] Remove horizontal scrollbar in Zeppeli...

Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1241 LGTM :smile: --- 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

[GitHub] zeppelin issue #1241: [ZEPPELIN-1217] Remove horizontal scrollbar in Zeppeli...

Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1241 Is this good to go 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

[GitHub] zeppelin pull request #1365: ZEPPELIN-1374. Should prevent use dot in interp...

GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/1365 ZEPPELIN-1374. Should prevent use dot in interpreter name ### What is this PR for? dot is invalid for interpreter name as it is used as the seprater of interpreter group name and interpreter

[GitHub] zeppelin issue #1365: ZEPPELIN-1374. Should prevent use dot in interpreter n...

Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1365 \cc @minahlee @jongyoul Ready for 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

[GitHub] zeppelin issue #1362: [ZEPPELIN-1178] Tooltip: Show chart type when hovering...

Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1362 +1 for @corneadoug, rest 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 feature

[GitHub] zeppelin issue #1364: [MINOR] Remove unnecessary question mark

Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1364 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 feature enabled and wishes so, or if

[GitHub] zeppelin issue #1353: ZEPPELIN-1326: make profile to select dependency of ha...

Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1353 @jongyoul does this looks OK ? --- 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] zeppelin issue #1360: [ZEPPELIN-1366] Removed legacy JDBC alias

Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1360 @jongyoul Then can we use JDBC interpreter as described in [How to use](http://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/interpreter/jdbc.html#how-to-use) section like before? I just wondered :D

[GitHub] zeppelin issue #1360: [ZEPPELIN-1366] Removed legacy JDBC alias

Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1360 @bzz @Leemoonsoo Thanks, I don't know the proper location to describe the changes actually. I'll also update upgrade.md. And about changing a behaviour of JDBC, in case of `%jdbc(mysql)`, I

[GitHub] zeppelin issue #1283: Buffer append output results + fix extra incorrect res...

Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1283 @corneadoug yes, that streams output every one seconds if i try the master branch. However, I did some more experiment and found Output streaming : master branch (X), this

[GitHub] zeppelin issue #1301: [MINOR] Removed unused profiles from spark/pom.xml

Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1301 @bzz Yes, what you mentioned is related to #1339 which will make a change to build custom Spark. --- If your project is set up for it, you can reply to this email and have your reply appear on

[jira] [Created] (ZEPPELIN-1374) Should prevent use dot in interpreter name

Jeff Zhang created ZEPPELIN-1374: Summary: Should prevent use dot in interpreter name Key: ZEPPELIN-1374 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1374 Project: Zeppelin Issue

[GitHub] zeppelin issue #1356: [ZEPPELIN-1346] Add contribution guidelines to the web...

Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1356 @felixcheung I agree with @corneadoug. If this document page need to be posted to the external resources, then we may need to attach `Apache` in front of every `Zeppelin`. But we don't need to

[GitHub] zeppelin issue #1362: [ZEPPELIN-1178] Tooltip: Show chart type when hovering...

Github user vensant commented on the issue: https://github.com/apache/zeppelin/pull/1362 Hi all, I will change the tool tip names as suggested. Thank you all Regards, Venkat On 26 Aug 2016 7:57 am, "Lee moon soo" wrote:

[GitHub] zeppelin issue #1339: [WIP][ZEPPELIN-1332] Remove spark-dependencies & sugge...

Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1339 @bzz Thank you for saying so! Then I'll continue my work in here and let you know :) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] zeppelin issue #1351: [ZEPPELIN/1356] The graph legend truncates at the near...

Github user Peilin-Yang commented on the issue: https://github.com/apache/zeppelin/pull/1351 @corneadoug sure, will do and push --- 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] zeppelin issue #1339: [WIP][ZEPPELIN-1332] Remove spark-dependencies & sugge...

Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1339 Thank you for kind explanation and feedback. I think you proposal and implementation with recent updates makes perfect sense. Please keep up a good work and ping me back for the final review,

[GitHub] zeppelin issue #1301: [MINOR] Removed unused profiles from spark/pom.xml

Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1301 Quick question: just to double-check - this should not affect users following published build instructions like https://www.mapr.com/blog/building-apache-zeppelin-mapr-using-spark-under-yarn ,

[GitHub] zeppelin issue #932: Replace CXF with Jersey2 [ZEPPELIN-903]

Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/932 After the licenses clarification, I think we should be good to merge - I love the idea of having Jersey2. \cc @Leemoonsoo @jongyoul for review as well --- If your project is set up for it,

[GitHub] zeppelin issue #932: Replace CXF with Jersey2 [ZEPPELIN-903]

Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/932 Sorry, been a busy week. I think it needs to be rebased on latest master. Few things on licensing: There are two same **CDDL 1.1** texts now

[GitHub] zeppelin issue #1362: [ZEPPELIN-1178] Tooltip: Show chart type when hovering...

Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1362 Tested, LGTM However maybe the tooltip names could be changed to something more simple: * Multi Bar Chart -> Bar Chart * Stacked Area Chart -> Area Chart * Line With Focus Chart

[GitHub] zeppelin issue #1351: [ZEPPELIN/1356] The graph legend truncates at the near...

Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1351 @Peilin-Yang make sense, have updated jira fix-for version. @corneadoug absolutely, please take your time for review. --- If your project is set up for it, you can reply to this email and

[GitHub] zeppelin issue #1351: [ZEPPELIN/1356] The graph legend truncates at the near...

Github user Peilin-Yang commented on the issue: https://github.com/apache/zeppelin/pull/1351 @bzz is that possible to apply this to 0.6.0-SNAPSHOT. We are still using that version in Twitter:) --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin issue #1364: [MINOR] Remove unnecessary question mark

Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1364 Thanks @zjffdu for taking care. LGTM and merge to master if there's no more discussion. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] zeppelin issue #1356: [ZEPPELIN-1346] Add contribution guidelines to the web...

Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1356 @felixcheung I don't think that adding Apache in front of every Zeppelin is necessary. Especially since this is documentation for the project itself. It is fine to use the shorten name

[GitHub] zeppelin issue #1360: [ZEPPELIN-1366] Removed legacy JDBC alias

Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1360 I think it changes way of configuration. From single jdbc interpreter setting manages multiple jdbc connection, to individual interpreter setting manage single jdbc connection. How

[GitHub] zeppelin issue #1362: [ZEPPELIN-1178] Tooltip: Show chart type when hovering...

Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1362 @vensant thank you for contributing improvement! With GUI changes, in my experience, adding simple screenshots to the PR itself (drag'n'drop to description) before\after usually helps to

[GitHub] zeppelin issue #1363: [ZEPPELIN-1371]add text/numeric conversion support to ...

Github user Peilin-Yang commented on the issue: https://github.com/apache/zeppelin/pull/1363 Well, consider it alone, yes it is. --- 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] zeppelin issue #1351: [ZEPPELIN/1356] The graph legend truncates at the near...

Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1351 @bzz I'd like to review it first --- 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] zeppelin issue #1351: [ZEPPELIN/1356] The graph legend truncates at the near...

Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1351 Looks great to me, thank you for taking care @Peilin-Yang ! Merging if there is no further discussion. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin issue #1360: [ZEPPELIN-1366] Removed legacy JDBC alias

Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1360 LGTM. > Is there breaking changes for older versions? Yes Do we have a change-log or something where we can keep track of breaking changes to highlight them in the release?

[GitHub] zeppelin issue #1357: ZEPPELIN-1364: Return only the ran paragraph details f...

Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1357 @vipul1409 thank you, it sounds interesting! Could you please let know, if it is ready for review? @Leemoonsoo @corneadoug do you guys see any downsides\side-effects for such change?

[GitHub] zeppelin issue #1363: [ZEPPELIN-1371]add text/numeric conversion support to ...

Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1363 Go it! It would be really nice to have such defaults for numbers, that make them sort numerically. As for letting user change it - it sounds good, am it looks like the main use case for it

[GitHub] zeppelin issue #1347: ZEPPELIN-1284. Unable to run paragraph with default in...

Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1347 LGTM. Merging if there is no further discussion --- 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] zeppelin issue #1363: [ZEPPELIN-1371]add text/numeric conversion support to ...

Github user Peilin-Yang commented on the issue: https://github.com/apache/zeppelin/pull/1363 @bzz Actually I am currently look into that and I think it is possible because numbers are automatically detected and that is why the commas are inserted. But I think even we could have

[GitHub] zeppelin issue #1283: Buffer append output results + fix extra incorrect res...

Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1283 @Leemoonsoo The behaviour is still the same with: ``` (1 to 40).foreach{ i=> Thread.sleep(1000) println("Hello " + i) } ``` --- If your project is set

[GitHub] zeppelin issue #1363: [ZEPPELIN-1371]add text/numeric conversion support to ...

Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1363 So cool - this should solve sorting issue for numeric columns! Thank you @Peilin-Yang! One question - do you guys think it could be possible to detect numeric columns automatically using

[GitHub] zeppelin issue #1363: [ZEPPELIN-1371]add text/numeric conversion support to ...

Github user Peilin-Yang commented on the issue: https://github.com/apache/zeppelin/pull/1363 @corneadoug thanks for your advice! I updated the contents of PR so that it includes the use cases of this change. Please let me know whether they make any sense. --- If your project

[GitHub] zeppelin issue #1358: [ZEPPELIN-1365] Error of Zeppelin Application in devel...

Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1358 re-trigger CI --- 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,

[GitHub] zeppelin pull request #1358: [ZEPPELIN-1365] Error of Zeppelin Application i...

Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1358 --- 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

[GitHub] zeppelin pull request #1358: [ZEPPELIN-1365] Error of Zeppelin Application i...

GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/1358 [ZEPPELIN-1365] Error of Zeppelin Application in development mode. ### What is this PR for? This PR fixes the bug of running application in development mode. ### What type

[GitHub] zeppelin pull request #1352: [ZEPPELIN-1327] Fix bug in z.show for Python in...

Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1352 --- 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

[GitHub] zeppelin issue #1363: [ZEPPELIN-1371]add text/numeric conversion support to ...

Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1363 @Peilin-Yang Thanks for the contribution. Small tip: Instead of hosting your screenshot, you can drop the image directly in the input of the PR description and the image will be hosted by

Re: Accessing bound vars from JS

There is no real API to access the value inside of JS. However, if we are really talking about accessing values set using z.angularBing(), you can do it in your case with: angular.element('#dummy').scope().valueBinded I have a small example here of an autocomplete input: *Notebook:*

[GitHub] zeppelin issue #1364: [MINOR] Remove unnecessary question mark

Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1364 \cc @Leemoonsoo --- 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

[GitHub] zeppelin pull request #1364: [MINOR] Remove unnecessary question mark

GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/1364 [MINOR] Remove unnecessary question mark ### What is this PR for? Just remove the unnecessary question mark. ### What type of PR is it? [Documentation] ### Todos

[GitHub] zeppelin pull request #1363: [ZEPPELIN-1371]add text/numeric conversion supp...

GitHub user Peilin-Yang reopened a pull request: https://github.com/apache/zeppelin/pull/1363 [ZEPPELIN-1371]add text/numeric conversion support to table display ### What is this PR for? We people in Twitter have great demand of adding a flexible drop down menu for the columns

[GitHub] zeppelin pull request #1363: [ZEPPELIN-1371]add text/numeric conversion supp...

Github user Peilin-Yang closed the pull request at: https://github.com/apache/zeppelin/pull/1363 --- 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

[jira] [Created] (ZEPPELIN-1372) sorting order in the table is always lexicographically even it shows as numbers

Peilin Yang created ZEPPELIN-1372: - Summary: sorting order in the table is always lexicographically even it shows as numbers Key: ZEPPELIN-1372 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1372

Accessing bound vars from JS

The Angular interpreter is very useful, but I'm having a difficult time using values from it inside custom JavaScript. Thus far, the only means I can find of accessing variables is something like: %angular var div = $('#dummy'); //Given an element in the note & list of values to fetch from

[GitHub] zeppelin pull request #1363: [ZEPPELIN-1371]add text/numeric conversion supp...

GitHub user Peilin-Yang opened a pull request: https://github.com/apache/zeppelin/pull/1363 [ZEPPELIN-1371]add text/numeric conversion support to table display ### What is this PR for? We people in Twitter have great demand of adding a flexible drop down menu for the columns in

[jira] [Created] (ZEPPELIN-1371) Add flexible text/numeric conversion dropdown menu on table display

Peilin Yang created ZEPPELIN-1371: - Summary: Add flexible text/numeric conversion dropdown menu on table display Key: ZEPPELIN-1371 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1371 Project:

[GitHub] zeppelin pull request #1295: [ZEPPELIN-1301] fix potential encoding problem ...

Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1295 --- 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

[GitHub] zeppelin issue #1323: [HOTFIX] Add logins authentication for hdfs file inter...

Github user rawkintrevo commented on the issue: https://github.com/apache/zeppelin/pull/1323 @cloverhearts Not going to pretend I'm an expert on HDFS- here's my use case, and I am happy to offer further explanation. I have a [BigInsights

[GitHub] zeppelin issue #1283: Buffer append output results + fix extra incorrect res...

Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1283 Text output streaming used to work in following way (branch-0.6) 

[GitHub] zeppelin issue #1357: ZEPPELIN-1364: Return only the ran paragraph details f...

Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/1357 Didn't try the changes but definitively +1 for this to enhance performance (tried to imagine side effects but didn't find any atm) --- If your project is set up for it, you can reply to this

[GitHub] zeppelin pull request #1362: [ZEPPELIN-1178] Tooltip: Show chart type when h...

GitHub user vensant opened a pull request: https://github.com/apache/zeppelin/pull/1362 [ZEPPELIN-1178] Tooltip: Show chart type when hovering over chart icon ### What is this PR for? A usability improvement: Added tooltips for the chart icons in Zeppelin paragraphs, showing

[GitHub] zeppelin pull request #1315: [ZEPPELIN-530] Added changes for Credential Pro...

Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1315 --- 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

[GitHub] zeppelin issue #1283: Buffer append output results + fix extra incorrect res...

Github user beriaanirudh commented on the issue: https://github.com/apache/zeppelin/pull/1283 @corneadoug @bzz this is ready for 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

[GitHub] zeppelin issue #932: Replace CXF with Jersey2 [ZEPPELIN-903]

Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/932 @bzz are we ok now with the license information? Travis is red for 3 on the 9 profiles, but this comes from a timeout trying to launch/access spark ```16/08/17 15:36:04 INFO

[GitHub] zeppelin pull request #1361: [ZEPPELIN-1354] [WIP] Inject Services

GitHub user echarles opened a pull request: https://github.com/apache/zeppelin/pull/1361 [ZEPPELIN-1354] [WIP] Inject Services ### What is this PR for? Services like authentication, authorization, security filters... should be injected rather than manually instanciated.