trinodb / trino

Official repository of Trino, the distributed SQL query engine for big data, formerly known as PrestoSQL (https://trino.io)
https://trino.io
Apache License 2.0
10.25k stars 2.95k forks source link

Move Metadata access logic from LogicalPlanner to Analyzer #13740

Open homar opened 2 years ago

homar commented 2 years ago

In LogicalPlanner while planning update metadata is called to obtain tableMetadata. This should be part of Analyzer not planner.

findepi commented 2 years ago

@djsstarburst is planning on fixing this together with https://github.com/trinodb/trino/issues/13621

djsstarburst commented 1 year ago

This issue is fixed by PR #13926, which is ready for review.