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

Adds an options to automatically clean MLSQLRest temp data #1778

Merged
merged 1 commit into from
May 10, 2022

Conversation

chncaesar
Copy link
Contributor

What changes were proposed in this pull request?

Adds an options to automatically clean temp data after job is finished. Option: config.enableRequestCleaner , false by default.

Usage:

LOAD Rest.`http://www.byzer.org/`
WHERE `config.connect-timeout`="10s"
AND `config.page.next`= "http://www.byzer.org/index={0}"
AND `config.method`="get"
AND `config.page.retry`="1"
AND `config.page.values`="offset:0,1"
AND `header.content-type`="application/json"
ANd `config.debug`="true"
AND `config.enableRequestCleaner`="true"
AS dd;

How was this patch tested?

Added a ut case: Temp data should be cleaned if config.enableRequestCleaner is true. unit test passed.
image

Are there and DOC need to update?

  • Will update mlsql-docs repo

Spark Core Compatibility

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.

3 participants