Skip to content

JAVA-1064: getTable create statement doesn't properly handle quotes in primary key.#582

Merged
olim7t merged 5 commits into
2.1from
java1064
Mar 3, 2016
Merged

JAVA-1064: getTable create statement doesn't properly handle quotes in primary key.#582
olim7t merged 5 commits into
2.1from
java1064

Conversation

@adutra
Copy link
Copy Markdown
Contributor

@adutra adutra commented Feb 2, 2016

Also fixes reserved keywords.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Small enhancement backported from 3.0: this line would throw an OutOfBoundsException for the empty string.

@adutra adutra force-pushed the java1064 branch 2 times, most recently from 70d47c1 to acfdd5b Compare February 2, 2016 14:45
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

for a better spacing when outputting columns in non-formatted mode.

@olim7t olim7t added this to the 2.1.10 milestone Feb 9, 2016
* @test_category metadata
*/
@Test(groups = "short")
public void should_parse_table_with_case_sensitive_column_names_and_reserved_keywords() throws Exception {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice!

@tolbertam
Copy link
Copy Markdown
Contributor

Looks good 👍

static final String STATEMENT_START = "\n\t";
static final String COLUMN_FORMATTING = "\n\t\t";

static final List<String> RESERVED_KEYWORDS = Arrays.asList(
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

replaced with the equivalent in Metadata.

olim7t added a commit that referenced this pull request Mar 3, 2016
JAVA-1064: getTable create statement doesn't properly handle quotes in primary key.
@olim7t olim7t merged commit cc4db43 into 2.1 Mar 3, 2016
@olim7t olim7t deleted the java1064 branch March 3, 2016 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants