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

Tracking issue of the first release of OpenDAL Java binding #2339

Closed
3 tasks done
tisonkun opened this issue May 27, 2023 · 4 comments · Fixed by #2352
Closed
3 tasks done

Tracking issue of the first release of OpenDAL Java binding #2339

tisonkun opened this issue May 27, 2023 · 4 comments · Fixed by #2352
Assignees

Comments

@tisonkun
Copy link
Member

tisonkun commented May 27, 2023

  • Improve Javadoc for all classes
  • Integrate Javadoc generation with official site CI workflow
  • Ensure the release process works
@tisonkun
Copy link
Member Author

@Xuanwo You may take a look at whether other stuff is missing here before we make a formal 0.1.0 release.

For 1 and 2, they should be trivial with mvn javadoc:javadoc and copy the generated content to the site folder, like other bindings do.

For 3, I'll investigate a bit if we can add an automation in ci-opendal repo now. The whole process should be associated with Nexus staging and close staging also.

@Xuanwo
Copy link
Member

Xuanwo commented May 27, 2023

Mostly LGTM. I have some ideas now, but I don't think they are significant enough to block opendal-java 0.1.

  • Does java have something like Reader/Writer? If so, it's better for us to implement them so that users can use our Reader like native File struct.
  • We can export append support for java which is needed by @ShadowySpirits
  • Additionally, maybe @ShadowySpirits has other features are required to initiate a PoC with opendal-java.

@tisonkun
Copy link
Member Author

Reader/Writer

I don't think so. If we further integrate with other interfaces or frameworks, we may do that. But Java doesn't provide such abstractions. InputStream/OutputStream may be, but I don't have a good idea here.

append

Can you share a link to the Rust append method?

@suyanhanx
Copy link
Member

Can you share a link to the Rust append method?

https://opendal.apache.org/docs/rust/opendal/struct.Operator.html#method.append

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

Successfully merging a pull request may close this issue.

3 participants