Github user Tagar commented on the issue:
https://github.com/apache/zeppelin/pull/2601
> I believe one could remove old button "Download Data as CSV".
filed https://issues.apache.org/jira/browse/ZEPPELIN-3511 to remove old
export csv/tsv
This idea was also briefly
Github user BruceXu1991 commented on the issue:
https://github.com/apache/zeppelin/pull/2601
@mebelousov OK, I will close this PR.
---
Github user mebelousov commented on the issue:
https://github.com/apache/zeppelin/pull/2601
https://github.com/apache/zeppelin/pull/2601
Hi, @BruceXu1991!
As @1ambda, I believe one could remove old button "Download Data as CSV".
The new exporter works fine and conform
Github user BruceXu1991 commented on the issue:
https://github.com/apache/zeppelin/pull/2601
@1ambda hi, is there any conclusion?
---
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/2601
@BruceXu1991 I think that's fair. @1ambda what do you think?
---
Github user BruceXu1991 commented on the issue:
https://github.com/apache/zeppelin/pull/2601
@felixcheung The
API(http://ui-grid.info/docs/#/tutorial/206_exporting_data) only support export
data as csv format but no tsv format supporting. So if using this API, tsv
format related PR
Github user BruceXu1991 commented on the issue:
https://github.com/apache/zeppelin/pull/2601
ok, I will do it.
---
Github user BruceXu1991 commented on the issue:
https://github.com/apache/zeppelin/pull/2601
@Tagar Yes, ZEPPELIN-2956 and ZEPPELIN-1803 point to the same issue.
RFC-4180 explains the issue.
---
Github user Tagar commented on the issue:
https://github.com/apache/zeppelin/pull/2601
I wonder if it's related to
https://issues.apache.org/jira/browse/ZEPPELIN-1803 ?
---
Github user BruceXu1991 commented on the issue:
https://github.com/apache/zeppelin/pull/2601
@1ambda If the table library in #2323 could deal with the problem of
downloading column value both contain delimiter and quotation mark, It may be
better to use library.
---
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2601
Hi, https://github.com/apache/zeppelin/pull/2323 added a new table library
and it supports downloads. (for 0.8.0-SNAPSHOT)
Github user BruceXu1991 commented on the issue:
https://github.com/apache/zeppelin/pull/2601
After merging to master, CI tests still have some exception, but the
exception seems have no relation with this patch.
> job 7.2
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2601
@BruceXu1991 Right, https://github.com/apache/zeppelin/pull/2603 fix CI job
6.4 error.
You can try merge master and see if CI becomes green!
---
Github user BruceXu1991 commented on the issue:
https://github.com/apache/zeppelin/pull/2601
I tryed again, but CI test failed job 6.4, and I could not figure out what
the error means.
https://travis-ci.org/BruceXu1991/zeppelin/jobs/281177302
---
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/2601
hmm, that's odd, I don't think I've seen that before perhaps.. try
again?
---
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/2601
could you setup CI tests
https://zeppelin.apache.org/contribution/contributions.html#continuous-integration
---
16 matches
Mail list logo