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

feat: make services-gdrive compile for wasm target #3808

Merged
merged 1 commit into from
Dec 23, 2023

Conversation

Young-Flash
Copy link
Member

@Young-Flash Young-Flash commented Dec 23, 2023

tracked in #3803

Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@Xuanwo Xuanwo merged commit cd323f8 into apache:main Dec 23, 2023
39 checks passed
@Young-Flash Young-Flash deleted the wasm-gdrive branch December 23, 2023 11:25
@Young-Flash
Copy link
Member Author

let mut cfg: Gdrive = Gdrive::default();
cfg.access_token("...").root("/tmp");

let op = Operator::new(cfg).unwrap().finish();
op.write(
    "test-gdrive-wasm",
    "Hello, WASM! We are from OpenDAL at rust side!"
        .as_bytes()
        .to_vec(),
)
.await
.unwrap();
let bs = op.read("test-gdrive-wasm").await.unwrap();

I am test gdrive over wasm, when I work with the above code, it failed with

image

but that code works well if I run it in rust side, am I missing something?

@Xuanwo
Copy link
Member

Xuanwo commented Dec 23, 2023

let mut cfg: Gdrive = Gdrive::default();

cfg.access_token("...").root("/tmp");



let op = Operator::new(cfg).unwrap().finish();

op.write(

    "test-gdrive-wasm",

    "Hello, WASM! We are from OpenDAL at rust side!"

        .as_bytes()

        .to_vec(),

)

.await

.unwrap();

let bs = op.read("test-gdrive-wasm").await.unwrap();

I am test gdrive over wasm, when I work with the above code, it failed with

image

but that code works well if I run it in rust side, am I missing something?

Would like to raise an new issue for this? We need go more deeper into this.

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

Successfully merging this pull request may close these issues.

2 participants