Skip to content

Conversation

@vvysotskyi
Copy link
Member

  • Made changes to pass resulting precision for VARCHAR, TIME and TIMESTAMP types to avoid the literal trimming during the cast removing.

For problem description please see DRILL-6959.

@arina-ielchiieva
Copy link
Member

@KazydubB could you please review?

@KazydubB
Copy link
Member

@vvysotskyi could you change commit message to reflect actual changes, please?
Other than that LGTM.

@vvysotskyi vvysotskyi changed the title DRILL-6959: Query with filter with cast to timestamp literal does not return any results DRILL-6959: Fix loss of precision when casting time and timestamp literals in filter condition Jan 11, 2019
@vvysotskyi
Copy link
Member Author

@KazydubB, done.

Copy link
Member

@KazydubB KazydubB left a comment

Choose a reason for hiding this comment

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

+1

@sohami sohami closed this in 06a210c Jan 12, 2019
lushuifeng pushed a commit to lushuifeng/drill that referenced this pull request Jun 21, 2019
xiangt920 pushed a commit to xiangt920/drill that referenced this pull request Dec 26, 2019
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.

3 participants