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

Support field separator for Hive TEXTFILE #1439

Merged

Conversation

kokosing
Copy link
Member

@kokosing kokosing commented Sep 4, 2019

Support field separator for Hive TEXTFILE

return getTextFormatProperty(tableProperties, TEXTFILE_FIELD_SEPARATOR_ESCAPE);
}

private static Optional<Character> getTextFormatProperty(Map<String, Object> tableProperties, String key)
Copy link
Member

Choose a reason for hiding this comment

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

getTextFormat*Separator*Property ?

also, isn't this equivalent to, but implemented differently than, getCsvSerdeProperty?


@Language("SQL") String createTableSql = format("" +
"CREATE TABLE %s.%s.test_create_external (\n" +
" name varchar\n" +
" action varchar,\n" +
" object varchar\n" +
Copy link
Member

Choose a reason for hiding this comment

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

object -> name

object may look like some weird data type

@kokosing kokosing force-pushed the origin/master/161_hive_field_separator branch from e22f096 to 6d248f6 Compare September 12, 2019 07:13
@kokosing kokosing merged commit 3e98801 into trinodb:master Sep 13, 2019
@kokosing kokosing deleted the origin/master/161_hive_field_separator branch September 13, 2019 06:45
@kokosing kokosing mentioned this pull request Sep 13, 2019
6 tasks
@martint martint added this to the 319 milestone Sep 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants