snowpark-python
snowpark-python copied to clipboard
SNOW-1060154: Add telemetry for query plan height and num duplicate subtrees
Please answer these questions before submitting your pull requests. Thanks!
-
What GitHub issue is this PR addressing? Make sure that there is an accompanying issue to your PR.
Fixes #SNOW-1060154
-
Fill out the following pre-review checklist:
- [x] I am adding a new automated test(s) to verify correctness of my new code
- [ ] I am adding new logging messages
- [x] I am adding a new telemetry message
- [ ] I am adding new credentials
- [ ] I am adding a new dependency
-
Please describe how your code solves the related issue.
This PR add
plan_height
andnum_duplicate_nodes
inSnowflakePlan
andSelectable
class to compute characteristics of the query plan tree and report these in telemetry
I assume it will be rebased after #1295 being merged?
I assume it will be rebased after #1295 being merged?
yes
Let's get Yijun's approval before merging
Client performance when running union api when 50 columns and varying n calls: