-
Notifications
You must be signed in to change notification settings - Fork 28.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[SPARK-7163] [SQL] minor refactory for HiveQl #5715
Conversation
new this.QueryExecution(plan) | ||
/* A catalyst metadata catalog that points to the Hive Metastore. */ | ||
@transient | ||
override protected[sql] lazy val catalog = new HiveMetastoreCatalog(this) with OverrideCatalog |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reorder to make catalog, functionRegistry, analyzer, sqlParser togethor
Test build #30962 has finished for PR 5715 at commit
|
Test build #31353 has finished for PR 5715 at commit
|
Test build #31503 has finished for PR 5715 at commit
|
We can consider breaking this up, but lets do it after the 1.4 release. Otherwise you are going to cause tons of conflicts with inflight PRs. |
yes, after related PR in such as windowfunction we can breaking this up |
Can we close this issue until you have time to bring it up to date? |
1/5 update:
A work to refactory hiveql to make it more clean and readable
1 move ASTNode related util method to a new object named HiveASTNodeUtil
2 delete no use method in HiveQl
3 override
sqlParser
in HiveQLDialec.4 rename HiveQl to HiveQlConverter
The code change seems very big, but actually it's the renaming of HiveQl and extracting some method to HiveASTNodeUtil cause that.
/cc @rxin