Skip to content

Sj test1#3

Open
sujay-jain wants to merge 8 commits into
masterfrom
sj_test1
Open

Sj test1#3
sujay-jain wants to merge 8 commits into
masterfrom
sj_test1

Conversation

@sujay-jain
Copy link
Copy Markdown
Owner

Please make sure your submission complies with our Development, Formatting, and Commit Message guidelines.

Fill in the release notes towards the bottom of the PR description.
See Release Notes Guidelines for details.

== RELEASE NOTES ==

General Changes
* ...
* ...

Hive Changes
* ...
* ...

If release note is NOT required, use:

== NO RELEASE NOTE ==

caithagoras0 and others added 8 commits March 3, 2020 18:36
- Fix the failure when column is null.
- Fix incorrect index in default result set converter.
- Allow result rows to be ignored.
Support determinism analysis for simple queries with top-level ORDER
BY LIMIT clause. Select, Insert, and CreateTableAsSelect are supported,
but only when the query part is a Select query, not a SetOperation(
i.e., Union, Intersect, and Except).

"INSERT INTO ...SELECT ... ORDER BY ... LIMIT ..." is supported.
"INSERT INTO ... SELECT ... UNION ALL ... SELECT ... ORDER BY ...
LIMIT ..." is not supported.

To check for determinism of ORDER BY LIMIT queries, we run the Select
query with limit N + 1, project all the necessary ORDER BY columns,
and check whether there is a tie on the ORDER BY columns for the
n-th row and the (n+1)-th row.
SMILE support has flaky edge cases, and we cache the
serialization now which already has reduced the cost.
Wait for final task info on abort to avoid missing task statistics in
case when query finishes before the final task info is received by the
TaskInfoFetcher.
Without this fix, the ConnectionSession was created without any session properties and hence the implementation which tried to use session properties failed.
Fix the problem downstream operators might receive LazyBlock after scan
introduced by prestodb#14169
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.

5 participants