[GitHub] incubator-zeppelin pull request: Added CSV and Tableau export func...

2016-05-09 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/725#issuecomment-217896651 @jmrr Yes, The goal was to take care of the export using #714 and #725 (for front and back-end export). This would allow most of the PR to be merged, a

[GitHub] incubator-zeppelin pull request: Added CSV and Tableau export func...

2016-05-09 Thread jmrr
Github user jmrr commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/725#issuecomment-217839838 I'm a bit confused. @corneadoug said in #761 that this PR, #725, will handle the export via a REST call through the `Settings` button. However @jongyoul mention

[GitHub] incubator-zeppelin pull request: Added CSV and Tableau export func...

2016-03-19 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/725#issuecomment-198150918 @markncooper Thanks for the contribution. If you're trying to drop limit of number of rows to be downloaded, how about following approach?

[GitHub] incubator-zeppelin pull request: Added CSV and Tableau export func...

2016-02-18 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/725#issuecomment-185983438 @markncooper @zhongneu Hello folks, this issue is handled by #6 initially. I think it's better that we focus one PR only. Do you mind improving #6 by commen

[GitHub] incubator-zeppelin pull request: Added CSV and Tableau export func...

2016-02-18 Thread zhongneu
Github user zhongneu commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/725#issuecomment-185934584 @markncooper Right, but if the export button is in the chart interface, shouldn't we only export the rows shown in UI? I also have a very similar ca

[GitHub] incubator-zeppelin pull request: Added CSV and Tableau export func...

2016-02-18 Thread markncooper
Github user markncooper commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/725#issuecomment-185918386 The big issue with simply exporting from the frontend is that you can only export as many rows as are displayed by the UI, right? I'd like to dump 1000s

[GitHub] incubator-zeppelin pull request: Added CSV and Tableau export func...

2016-02-18 Thread zhongneu
Github user zhongneu commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/725#issuecomment-185906306 @markncooper I am working on a pull request for this: https://github.com/apache/incubator-zeppelin/pull/714 I think we can directly export the data

[GitHub] incubator-zeppelin pull request: Added CSV and Tableau export func...

2016-02-18 Thread markncooper
GitHub user markncooper opened a pull request: https://github.com/apache/incubator-zeppelin/pull/725 Added CSV and Tableau export functionality ** What is this PR for? Allow users to export data in a paragraph to a CSV and / or Tableau via Tableau's WDC feature **