presto icon indicating copy to clipboard operation
presto copied to clipboard

[WIP] Fix plan checker plugin

Open pdabre12 opened this issue 6 months ago • 1 comments

Description

Motivation and Context

Impact

Test Plan

Contributor checklist

  • [ ] Please make sure your submission complies with our contributing guide, in particular code style and commit standards.
  • [ ] PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced.
  • [ ] Documented new properties (with its default value), SQL syntax, functions, or other functionality.
  • [ ] If release notes are required, they follow the release notes guidelines.
  • [ ] Adequate tests were added if applicable.
  • [ ] CI passed.

Release Notes

Please follow release notes guidelines and fill in the release notes below.

== RELEASE NOTES ==

General Changes
* ... 
* ... 

Hive Connector Changes
* ... 
* ... 

If release note is NOT required, use:

== NO RELEASE NOTE ==

pdabre12 avatar Jun 19 '25 04:06 pdabre12

CLA Signed

The committers listed above are authorized under a signed CLA.

  • :white_check_mark: login: pdabre12 / name: Pratik Joseph Dabre (7446d250da5d878ee65402d858bb3ae94f6321e3, 54629b6c5a2f75d9ed2a76efc084faf4ffe77601)

LGTM % the commits that introduce logging

tdcmeehan avatar Jun 20 '25 18:06 tdcmeehan

@tdcmeehan Removed the logging commits.

pdabre12 avatar Jun 20 '25 23:06 pdabre12