Skip to content

Add release notes for 0.245#15499

Merged
wenleix merged 2 commits intoprestodb:masterfrom
aweisberg:release-notes-0.245
Dec 16, 2020
Merged

Add release notes for 0.245#15499
wenleix merged 2 commits intoprestodb:masterfrom
aweisberg:release-notes-0.245

Conversation

@caithagoras
Copy link
Contributor

@caithagoras caithagoras commented Dec 4, 2020 •
edited by aweisberg
Loading

Missing Release Notes

Mayank Garg

Extracted Release Notes

All Commits

  • b2c7ab8 Load function managers for Presto-on-Spark driver only (Vic Zhang)
  • 8e1384e Add query schema to ConnectorSession (Mayank Garg)
  • 27ab25b Fix tests for prefer_manifests_to_list_files property (Nikhil Collooru)
  • 996738a Fix NPE when a FlatMap column is all empty (Bhavani Hari)
  • 388055f Add more filter rates to BenchmarkSelectiveStreamReaders (Ying Su)
  • 297b089 Improve reading the lengthVector in SliceDirectSelectiveStreamReader (Ying Su)
  • 02e0981 Improve reading unbounded varchar with batch read mode (Ying Su)
  • dc2d50b Make query result HTTP compression server configurable (James Petty)
  • d0c7393 Make query result HTTP compression client configurable (James Petty)
  • 1e00009 Optimize array_intersect by using OptimizedTypedSet (Ying Su)
  • 87c7766 Optimize array_except by using OptimizedTypedSet (Ying Su)
  • aabf9d6 Optimize array_union using OptimizedTypedSet (Ying Su)
  • 23e1ba7 Optimize map_concat using OptimizedTypedSet (Ying Su)
  • 30cacee Introducing OptimizedTypedSet (Ying Su)
  • ec7336d Add self join and table scan assignment tests for dereference pushdown (Beinan Wang)
  • 1305272 Fix base column name not present error when dereference pushdown enabled (Beinan Wang)
  • 1a804ac Fixes for prefer_manifests_to_list_files property (Nikhil Collooru)
  • cba87b4 Add syntax support for CREATE MATERIALIZED VIEW (Ge Gao)
  • fe1bc57 Add verifier flag for removing memory related session property (Wenlei Xie)
  • ca64162 Update to airlift 0.198 and airbase 100 (Leiqing Cai)
  • 7b41e69 Add session property manifest_verification_enabled (Nikhil Collooru)
  • ce41e2f Add PartitionLoader to load partitions (Nikhil Collooru)
  • 04c9ef6 Add new session property prefer_manifests_to_list_files (Nikhil Collooru)
  • 951ec9d Build presto-docs in Travis (Leiqing Cai)
  • 1918b62 Do not treat warning as errors when making presto-docs (Leiqing Cai)
  • 817d50a Fix split cacheable setting for affinity scheduling (Ke Wang)
  • c0e24ec Add UDF to map enum value to key (Daniel Ohayon)
  • 59e88dc Fix LongEnum deserialization in map keys (Daniel Ohayon)
  • 9d5db3c Fix extracting logic in dynamic filtering when integrated with filter pushdown (Ke Wang)
  • c96938b Store file names and sizes in partition parameters (Nikhil Collooru)
  • 36fdd40 Add getFileSize() to retrieve the size of the file (Nikhil Collooru)
  • faee0a0 Add filter pushdown tests for native worker (Masha Basmanova)
  • 566b0d8 Change task threshold revoking config type to DataSize (Saksham Sachdev)
  • 0bb6d8d Null unused array elements in SpillableFinalOnlyGroupedAccumulator (Saksham Sachdev)
  • 2e67116 Use serialized plan for fragment result caching (Shixuan Fan)
  • 15b0ea2 Minor refactor to SingleStreamSpillerChoice (Wenlei Xie)
  • aef9bfc Fix not registering TempStorage in PluginManager (Wenlei Xie)
  • 6970b6d Refactor TempStorage related interfaces (Wenlei Xie)
  • 548457a Pass in SqlParserOptions to HttpRequestSessionContext for Prepared Statements. (Dong Shi)
  • 2499b40 Add per-node revocable memory limit (Saksham Sachdev)
  • 32e32af Add warning for Presto on Spark (Vic Zhang)
  • 68b3c78 Reuse immutable partition fields in Glue partition conversion (James Petty)
  • 1b84222 Add pre-read hueristic to parquet MetadataReader (James Petty)
  • b23ce2e Remove spill-enabled requirement for enabling join spill (Rebecca Schlussel)
  • b2208ec Fix the check to correctly capture NULL positions (Bhavani Hari)
  • 63d85a3 Add ParametricType APIs to FunctionNamespaceManager (Rongrong Zhong)
  • a6d77a6 Add QualifiedObjectName to TypeSignature (Rongrong Zhong)
  • b411d77 Merge BuiltInTypeRegistry and BuiltinFunctionNamespaceManager (Rongrong Zhong)
  • 0ffe6df Move QualifiedObjectName to presto-common (Rongrong Zhong)
  • bde30f6 Refactor type coercion logic to TypeCoercer (Rongrong Zhong)
  • d8da5bc Rename TypeRegistry to BuiltInTypeRegistry (Rongrong Zhong)
  • cd44d24 Clean up TypeManager APIs (Rongrong Zhong)
  • ae66246 Remove Metadata.getTypeManager (Rongrong Zhong)
  • e24dcc2 Support function namespace manager for Presto-on-Spark (Vic Zhang)
  • be1e61c Refactor StaticFunctionNamespaceStore to load properties map (Vic Zhang)
  • 11535d4 Chage Block's toString to be unique with respect to hashCode (Rongrong Zhong)
  • 5e1687b Fix LambdaDefinitionExpression canonicalization (Rongrong Zhong)
  • 455cd6c Load configuration manager for Presto-on-Spark driver (Vic Zhang)

@aweisberg
Copy link
Contributor

aweisberg commented Dec 4, 2020

@pettyjamesm @shixuan-fan The release note for #15393 doesn't follow the release note guidelines here https://github.com/prestodb/presto/wiki/Release-Notes-Guidelines#phrasing
Can you please updated it to include details about the configuration options following those guidelines?

@aweisberg
Copy link
Contributor

aweisberg commented Dec 4, 2020

@caithagoras @daniel-ohayon Can you please add more detail about #15444 ?
Maybe the name of the UDF, possibly the parameter and return values?

@aweisberg aweisberg force-pushed the release-notes-0.245 branch from 39ecd8e to caf1271 Compare December 4, 2020 17:15
@pettyjamesm
Copy link
Contributor

pettyjamesm commented Dec 4, 2020

@pettyjamesm @shixuan-fan The release note for #15393 doesn't follow the release note guidelines here https://github.com/prestodb/presto/wiki/Release-Notes-Guidelines#phrasing
Can you please updated it to include details about the configuration options following those guidelines?

Updated the PR release notes section to mention the specific CLI flag and server side configuration property.

@aweisberg aweisberg force-pushed the release-notes-0.245 branch from 36171f9 to e9fc296 Compare December 4, 2020 22:03
@daniel-ohayon
Copy link
Contributor

daniel-ohayon commented Dec 4, 2020

@caithagoras @daniel-ohayon Can you please add more detail about #15444 ?
Maybe the name of the UDF, possibly the parameter and return values?

Updated the PR release note with the function signature

rschlussel
rschlussel reviewed Dec 8, 2020
View reviewed changes
mayankgarg1990
mayankgarg1990 suggested changes Dec 8, 2020
View reviewed changes
presto-docs/src/main/sphinx/release/release-0.245.rst Outdated
Copy link

@mayankgarg1990 mayankgarg1990 Dec 8, 2020

Choose a reason for hiding this comment

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

  • Add :func:enum_key to get the key corresponding to an enum value

Copy link
Contributor

@aweisberg aweisberg Dec 8, 2020

Choose a reason for hiding this comment

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

Are you advocating for not showing the signature?

presto-docs/src/main/sphinx/release/release-0.245.rst Outdated
Copy link

@mayankgarg1990 mayankgarg1990 Dec 8, 2020

Choose a reason for hiding this comment

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

``Improve parquet metadata reader by preloading data and reducing the number of reads` - I don't think we should dig too much into the implementation details. We should just link to the issue/pr

Copy link
Contributor

@aweisberg aweisberg Dec 8, 2020

Choose a reason for hiding this comment

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

The release note generation logic doesn't link to PRs. I added the link for this one, but generally what are we doing? Seems like we should do it automatically.

rschlussel
rschlussel reviewed Dec 8, 2020
View reviewed changes
Copy link
Contributor

@rschlussel rschlussel left a comment

Choose a reason for hiding this comment

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

squash all the fixes into the 0.245 release notes commit.

mayankgarg1990
mayankgarg1990 approved these changes Dec 8, 2020
View reviewed changes
caithagoras
caithagoras commented Dec 8, 2020
View reviewed changes
@caithagoras
Copy link
Contributor Author

caithagoras commented Dec 8, 2020

lgtm

@aweisberg aweisberg force-pushed the release-notes-0.245 branch from 3644cb3 to d6f6870 Compare December 9, 2020 01:41
@aweisberg
Copy link
Contributor

aweisberg commented Dec 9, 2020

@rongrong tagging you to review the warning I added.

rschlussel
rschlussel reviewed Dec 9, 2020
View reviewed changes
@aweisberg aweisberg force-pushed the release-notes-0.245 branch from d6f6870 to cbfd2bd Compare December 9, 2020 21:52
@rschlussel
Copy link
Contributor

rschlussel commented Dec 9, 2020

@aweisberg docs build is failing.

caithagoras
caithagoras commented Dec 14, 2020
View reviewed changes
presto-docs/src/main/sphinx/release/release-0.245.rst Outdated
Copy link
Contributor Author

@caithagoras caithagoras Dec 14, 2020

Choose a reason for hiding this comment

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

Missing period at the end of the line.

@caithagoras
Copy link
Contributor Author

caithagoras commented Dec 14, 2020 •
edited
Loading

There are still Travis build failure due to Sphinx syntax. You can figure out why it is failing by

cd presto-docs
make clean
make html

caithagoras
caithagoras commented Dec 14, 2020
View reviewed changes
presto-docs/src/main/sphinx/release/release-0.245.rst Outdated
Copy link
Contributor Author

@caithagoras caithagoras Dec 14, 2020

Choose a reason for hiding this comment

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

I ran make with Sphinx 2.4.0 and it gives me warning about this line.

WARNING: py:func reference target not found: enum_key

We either need to document the function, or do

``enum_key``

as a temporary workaround to not reference the function.

presto-docs/src/main/sphinx/release/release-0.245.rst Outdated
Copy link
Contributor Author

@caithagoras caithagoras Dec 14, 2020

Choose a reason for hiding this comment

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

Same here.

@wenleix wenleix merged commit 072f022 into prestodb:master Dec 16, 2020
@wenleix
Copy link
Contributor

wenleix commented Dec 16, 2020

Merged #15499, thanks for the contribution!

@caithagoras caithagoras mentioned this pull request Jan 11, 2021
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rschlussel rschlussel rschlussel approved these changes

@rongrong rongrong Awaiting requested review from rongrong

+4 more reviewers

@aweisberg aweisberg aweisberg left review comments

@daniel-ohayon daniel-ohayon daniel-ohayon left review comments

@wenleix wenleix wenleix approved these changes

@mayankgarg1990 mayankgarg1990 mayankgarg1990 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@caithagoras @aweisberg @pettyjamesm @daniel-ohayon @rschlussel @wenleix @mayankgarg1990 @bhhari