Skip to content

[branch-0.6] Take care of comma/tab escape in csv/tsv download - #1484

Closed
minahlee wants to merge 4 commits into
apache:branch-0.6from
minahlee:branch-0.6_ZEPPELIN-1001
Closed

[branch-0.6] Take care of comma/tab escape in csv/tsv download#1484
minahlee wants to merge 4 commits into
apache:branch-0.6from
minahlee:branch-0.6_ZEPPELIN-1001

Conversation

@minahlee

Copy link
Copy Markdown
Member

What is this PR for?

This PR backports #1445 to branch-0.6 since using ./dev/test_zeppelin_pr.py throws alert for backporting it.

What type of PR is it?

Bug Fix

@minahlee
minahleeforce-pushed the branch-0.6_ZEPPELIN-1001 branch from 33d447c to 58fda8eCompareOctober 5, 2016 06:36
@Leemoonsoo

Copy link
Copy Markdown
Member

LGTM

@minahlee

Copy link
Copy Markdown
MemberAuthor

Merging it into branch-0.6

asfgit pushed a commit that referenced this pull request Oct 6, 2016
### What is this PR for?
This PR backports #1445 to branch-0.6 since using `./dev/test_zeppelin_pr.py` throws alert for backporting it.
### What type of PR is it?
Bug Fix
Author: meenakshisekar <meenakshi.chandrasekar@imaginea.com>
Closes#1484 from minahlee/branch-0.6_ZEPPELIN-1001 and squashes the following commits:
58fda8e [meenakshisekar] Formatting errors corrected for build
f5c7f61 [meenakshisekar] Changes committed as per review comments in PR
e404075 [meenakshisekar] Altered the code as per PR suggestion 1465
574803c [meenakshisekar] Zeppelin-1001 Modified the data with comma/tab to be surronded by double quotes so that they are escaped.
@minahleeminahlee closed this Oct 6, 2016
@minahlee
minahlee deleted the branch-0.6_ZEPPELIN-1001 branch October 7, 2016 07:14
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@minahlee@Leemoonsoo@meenakshisekar