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

Javadoc issues #10378

Open
manuzhang opened this issue May 25, 2024 · 16 comments
Open

Javadoc issues #10378

manuzhang opened this issue May 25, 2024 · 16 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@manuzhang
Copy link
Collaborator

manuzhang commented May 25, 2024

Apache Iceberg version

main (development)

Query engine

None

Please describe the bug 🐞

There are several issues with Javadoc

  1. https://iceberg.apache.org/javadoc/nightly is not found
  2. Links in https://iceberg.apache.org/docs/1.4.3/metrics-reporting points to 1.5.2 Javadoc
  3. https://iceberg.apache.org/docs/javadoc/1.5.2/org/apache/iceberg/rest/RESTMetricsReporter.html is missing (fixed in Docs: Point links in metrics-reporting.md to GitHub Java source #10397)
@manuzhang manuzhang added the bug Something isn't working label May 25, 2024
@jbonofre
Copy link
Member

Good catch @manuzhang ! Let me fix that on the website.

@manuzhang
Copy link
Collaborator Author

@jbonofre Are you working on all these issues?

@jbonofre
Copy link
Member

I didn't start yet. Up to you. I can start on it tonight or you can take it.

@Fokko
Copy link
Contributor

Fokko commented May 28, 2024

Thanks for caching these @manuzhang. Let me know if you need any help!

@manuzhang
Copy link
Collaborator Author

Submitted #10397 for No. 3.

@Fokko
Copy link
Contributor

Fokko commented May 30, 2024

@manuzhang
Copy link
Collaborator Author

manuzhang commented May 30, 2024

It looks like the nightly points to latest according to https://github.com/apache/iceberg/tree/main/site#building-the-versioned-docs

It's documented but not actually implemented.

@Fokko
Copy link
Contributor

Fokko commented May 30, 2024

Links in https://iceberg.apache.org/docs/1.4.3/metrics-reporting points to 1.5.2 Javadoc

That one is quite tricky. It looks like it uses the macro plugin to set the version. The problem is that the macro plugin doesn't work well with the monorepo plugin. There is no way to override the version. I think the easiest way to fix this is to change the directory structure from:

docs/1.4.3/
javadoc/1.4.3/

to:

1.4.3/docs/
1.4.3/javadoc/

This way we can make relative links. I can play around with this tomorrow

@Fokko Fokko modified the milestones: Iceberg 1.6.0, Iceberg 1.7.0 Jul 12, 2024
@Fokko
Copy link
Contributor

Fokko commented Jul 12, 2024

I'll bump this to 1.7.0. It would be best if we could fix this using the macro plugin in combination with the monorepo plugin: backstage/mkdocs-monorepo-plugin#130

@jbonofre
Copy link
Member

@Fokko thanks ! It makes sense. I will take a look on this one for 1.7.0 (definitely not a blocker for 1.6.0).

@RussellSpitzer
Copy link
Member

@jbonofre Did you want to work on this?

@jbonofre
Copy link
Member

@RussellSpitzer sure thing ! I will ! Thanks !

@jbonofre
Copy link
Member

Sorry I forgot this one. I can take a look later tonight or tomorrow morning my time.

@manuzhang
Copy link
Collaborator Author

@jbonofre Have you got a chance to fix javadoc (issue No.1)?

@jbonofre jbonofre self-assigned this Nov 12, 2024
@Fokko Fokko modified the milestones: Iceberg 1.7.0, Iceberg 1.8.0 Nov 13, 2024
@jbonofre
Copy link
Member

I took a quick look on (1): I think the only thing missing is to add in the site/dev/common.sh a symbolic link nightly on the javadoc version fetch by git worktree add -f docs/javadoc. Basically, symlink on the latest javadoc fetch version with nightly.
I'm testing locally, I will open a PR soon.

@jbonofre
Copy link
Member

I'm resuming work on this one to include 1.7.2 and 1.8.0.

@jbonofre jbonofre modified the milestones: Iceberg 1.8.0, Iceberg 1.7.2 Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants