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

refactor: Use seperate Error instead of std::io::Error to avoid confusing #976

Merged
merged 25 commits into from
Nov 21, 2022
Merged

refactor: Use seperate Error instead of std::io::Error to avoid confusing #976

merged 25 commits into from
Nov 21, 2022

Conversation

Xuanwo
Copy link
Member

@Xuanwo Xuanwo commented Nov 21, 2022

I hereby agree to the terms of the CLA available at: https://databend.rs/dev/policies/cla/

Summary

refactor: Use seperate Error instead of std::io::Error to avoid confusing

@Xuanwo
Save work
11eb00b 
Signed-off-by: Xuanwo <[email protected]>
@Xuanwo
Save current work
167c26b 
Signed-off-by: Xuanwo <[email protected]>
@Xuanwo
Save work
724d77c 
Signed-off-by: Xuanwo <[email protected]>
@Xuanwo
Save work
c2ec9da 
Signed-off-by: Xuanwo <[email protected]>
@Xuanwo
Savr work
8bcb1c2 
Signed-off-by: Xuanwo <[email protected]>
@Xuanwo
Save work
c42bd2b 
Signed-off-by: Xuanwo <[email protected]>
@Xuanwo
Save work
170649f 
Signed-off-by: Xuanwo <[email protected]>
@Xuanwo
Save work
21b1caa 
Signed-off-by: Xuanwo <[email protected]>
@Xuanwo
Save work
03ad432 
Signed-off-by: Xuanwo <[email protected]>
@Xuanwo
Save work
360b2de 
Signed-off-by: Xuanwo <[email protected]>
@Xuanwo
It works
4350022 
Signed-off-by: Xuanwo <[email protected]>
@Xuanwo
Format code
c4cf881 
Signed-off-by: Xuanwo <[email protected]>
@Xuanwo
Merge remote-tracking branch 'origin/main' into refactor-error
b3f1920
@Xuanwo
Fix test
424dc78 
Signed-off-by: Xuanwo <[email protected]>
@Xuanwo
Save work
f611092 
Signed-off-by: Xuanwo <[email protected]>
@Xuanwo
Implement all functions for wrapper
5561cca 
Signed-off-by: Xuanwo <[email protected]>
@Xuanwo
Add wrapper for all backend
5ae00d7 
Signed-off-by: Xuanwo <[email protected]>
@Xuanwo
Remove not needed context
34decd6 
Signed-off-by: Xuanwo <[email protected]>
@Xuanwo
Remove not needed context
a2d1e43 
Signed-off-by: Xuanwo <[email protected]>
@Xuanwo
Update docs for error
ce46759 
Signed-off-by: Xuanwo <[email protected]>
@Xuanwo
Remove extra covert
7f60a93 
Signed-off-by: Xuanwo <[email protected]>
@Xuanwo Xuanwo marked this pull request as ready for review November 21, 2022 08:00
@Xuanwo
Fix build
8ab760e 
Signed-off-by: Xuanwo <[email protected]>
@Xuanwo
Fix test
0238a62 
Signed-off-by: Xuanwo <[email protected]>
@Xuanwo
Fix metadata
1d7db8f 
Signed-off-by: Xuanwo <[email protected]>
@Xuanwo
Fix unit test
b9a3b1d 
Signed-off-by: Xuanwo <[email protected]>
@Xuanwo Xuanwo merged commit 7f7b88b into main Nov 21, 2022
@Xuanwo Xuanwo deleted the refactor-error branch November 21, 2022 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Xuanwo