Github user wzhfy commented on the issue:
https://github.com/apache/spark/pull/15090
This pr has been updated based on all the above comments, changes are as
follows:
1. Modify analyze syntax a little bit: `identifierSeq` is now non-optional,
i.e. users must specify column names after `FOR COLUMNS`.
2. Check column correctness based on case sensitivity.
3. Deduplicate columns when checking correctness.
4. Support analyzing columns independently, i.e. when analyzing new
columns, now we donât remove stats of columns which are analyzed before.
5. Rename `BasicColStats` to `ColumnStats`.
6. Use 3*standard deviation to check ndv result in test cases.
7. Code refactoring based on comments.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
