Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[hotfix][hdfs]when writing to hfds,null value is set to '\N' #1408

Merged
merged 1 commit into from
Nov 22, 2022
Merged

[hotfix][hdfs]when writing to hfds,null value is set to '\N' #1408

merged 1 commit into from
Nov 22, 2022

Conversation

conghe2402
Copy link
Contributor

fix the issue: The hdfs connector cannot read correctly from the hfds files which generated by the hdfs connector itself.
When hdfs connector writing data to hdfs, null value of a column is set as a member of a String array that is a data row`s output, eg, data[0] = null, that results in "null" string value would be written into the hdfs text files.

solution:
null value is set to '\N' for hive specification so that hdfs connector can read from the output.

[hotfix][hdfs]when writing to hdfs, null value is set to '\N' for hive specification so that hdfs connector can read from the output

[hotfix][hdfs]when writing to hdfs, null value is set to '\N' for hive specification so that hdfs connector can read from the output
Copy link
Member

@FlechazoW FlechazoW left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Copy link
Contributor

@Paddy0523 Paddy0523 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@FlechazoW FlechazoW merged commit a924b98 into DTStack:master Nov 22, 2022
ll076110 pushed a commit that referenced this pull request Mar 3, 2023
[hotfix][hdfs]when writing to hdfs, null value is set to '\N' for hive specification so that hdfs connector can read from the output

(cherry picked from commit a924b98)
Sign up for free to 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.

None yet

3 participants