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

Explicit support for casting from Parquet short decimal to Presto ints #2451

Merged
merged 3 commits into from
Jan 13, 2020

Conversation

sopel39
Copy link
Member

@sopel39 sopel39 commented Jan 9, 2020

No description provided.

@cla-bot cla-bot bot added the cla-signed label Jan 9, 2020
@@ -870,6 +873,14 @@ public void testDecimalBackedByINT64()
}
}

@Test(expectedExceptions = ParquetDecodingException.class, expectedExceptionsMessageRegExp = "Presto decimal column type has different scale \\(2\\) than Parquet decimal column \\(1\\)")
Copy link
Member

Choose a reason for hiding this comment

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

You can use \\Q ... \\E to avoid having to escape within the literal string you're trying to match.

Copy link
Member

Choose a reason for hiding this comment

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

i'd recommend org.assertj.core.api.Assertions#assertThatThrownBy instead

@findepi findepi removed their request for review January 10, 2020 09:31
@sopel39 sopel39 merged commit 89a4ec4 into trinodb:master Jan 13, 2020
@sopel39 sopel39 deleted the ks/parquet_decimal_casting branch January 13, 2020 10:05
@sopel39 sopel39 mentioned this pull request Jan 13, 2020
7 tasks
@findepi findepi added this to the 329 milestone Jan 23, 2020
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.

None yet

3 participants