Skip to content
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

convert options in include statment to set statments #1633

Merged
merged 3 commits into from
Dec 23, 2021
Merged

Conversation

allwefantasy
Copy link
Contributor

@allwefantasy allwefantasy commented Dec 21, 2021

What changes were proposed in this pull request?

Related Issue

How was this patch tested?

pr-1633.mlsql

Are there and DOC need to update?

  • Doc is finished

Spark Core Compatibility

@allwefantasy allwefantasy merged commit 04217f9 into master Dec 23, 2021
@allwefantasy allwefantasy deleted the issue-1628 branch December 23, 2021 06:49
ckeys pushed a commit to ckeys/mlsql that referenced this pull request Dec 30, 2021
* convert options in include statment to set statments

* add test and fix exception throws when no suffix in  include statement

* ignore result check in pr_1633.mlsql

revise the code example

add checking multi group params

revise the remind msg when setting multi-group params in the Discretizer et

rename the checking functions
ckeys pushed a commit to ckeys/mlsql that referenced this pull request Dec 30, 2021
* convert options in include statment to set statments

* add test and fix exception throws when no suffix in  include statement

* ignore result check in pr_1633.mlsql

revise the code example

add checking multi group params

revise the remind msg when setting multi-group params in the Discretizer et

rename the checking functions
ckeys pushed a commit to ckeys/mlsql that referenced this pull request Jan 4, 2022
* convert options in include statment to set statments

* add test and fix exception throws when no suffix in  include statement

* ignore result check in pr_1633.mlsql

revise the code example

add checking multi group params

revise the remind msg when setting multi-group params in the Discretizer et

rename the checking functions
allwefantasy added a commit that referenced this pull request Jan 11, 2022
* convert options in include statment to set statments (#1633)

* convert options in include statment to set statments

* add test and fix exception throws when no suffix in  include statement

* ignore result check in pr_1633.mlsql

revise the code example

add checking multi group params

revise the remind msg when setting multi-group params in the Discretizer et

rename the checking functions

* add register example for the discretizer et

Co-authored-by: WilliamZhu <allwefantasy@gmail.com>
ckeys pushed a commit to ckeys/mlsql that referenced this pull request Jan 23, 2022
* convert options in include statment to set statments

* add test and fix exception throws when no suffix in  include statement

* ignore result check in pr_1633.mlsql

revise the code example

add checking multi group params

revise the remind msg when setting multi-group params in the Discretizer et

rename the checking functions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants