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.

Sign up for GitHub

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

Jump to bottom

fix databricks dbfs file path #4674

Merged
merged 3 commits into from
May 31, 2022
Merged

fix databricks dbfs file path #4674

merged 3 commits into from
May 31, 2022

Conversation

Le-Zheng
Copy link
Contributor

No description provided.

@Le-Zheng Le-Zheng changed the title fix databricks dbfs file path in save pickle fix databricks dbfs file path May 25, 2022
@Le-Zheng Le-Zheng mentioned this pull request May 25, 2022
Open
@Le-Zheng
Copy link
Contributor Author

verified on databricks:
ｔｒａｉｎ－ｄｂｆｓ

ｄａｔａｂｒｉｃｋｓ－ｆｉｌｅｓｙｓｔ

@Le-Zheng Le-Zheng requested a review from yangw1234 May 25, 2022 19:19
@yangw1234
Copy link
Contributor

LGTM
It would be nice to some unit tests on it though.

@Le-Zheng
Copy link
Contributor Author

LGTM It would be nice to some unit tests on it though.

ok. ut is added. jenkins passed: http://10.112.231.51:18888/job/BigDL-Orca-PR-Validation/691/ + http://10.112.231.51:18888/job/BigDL-PRVN-ray-ctx-example/146/

@Le-Zheng Le-Zheng merged commit 4182f05 into intel-analytics:main May 31, 2022
@Le-Zheng Le-Zheng deleted the fixdatabricksdbfs branch May 31, 2022 00:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yangw1234 yangw1234 Awaiting requested review from yangw1234

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Le-Zheng @yangw1234