Conversation
- Refactor website structure: - Rename `index.md` to `contributing.md` and update title. - Create new `index.md` as landing page. - Create `getting-started.md` with installation instructions. - Add Apache Iceberg C++ 0.2.0 release information. - Update navigation in `mkdocs.yml`. Co-authored-by: wgtmac <4684607+wgtmac@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
The build was failing in CI because `https://dlcdn.apache.org/arrow/apache-arrow-nanoarrow-0.7.0/apache-arrow-nanoarrow-0.7.0.tar.gz` returned a 404 error. This suggests the artifact has been removed from the mirror network (likely moved to archive). This commit updates `cmake_modules/IcebergThirdpartyToolchain.cmake` to include fallback URLs for `nanoarrow`: 1. The Apache dynamic download link (`dyn/closer.lua`). 2. The original `dlcdn` link (kept as one of the options). 3. The `archive.apache.org` link (stable location for older releases). 4. The GitHub release artifact link. This ensures the build can proceed even if the primary mirror is missing the file. Co-authored-by: wgtmac <4684607+wgtmac@users.noreply.github.com>
…oarrow URL Docs changes: - Refactor website structure: - Rename `index.md` to `contributing.md` and update title. - Create new `index.md` as landing page. - Create `getting-started.md` with installation instructions. - Add Apache Iceberg C++ 0.2.0 release information. - Update navigation in `mkdocs.yml`. Build fix: - Update `cmake_modules/IcebergThirdpartyToolchain.cmake` to include fallback URLs for `nanoarrow` download. - This fixes CI failures where the primary `dlcdn` mirror returns 404 for the nanoarrow tarball. - Fallbacks added: Apache dynamic downloader, archive.apache.org, and GitHub releases. Co-authored-by: wgtmac <4684607+wgtmac@users.noreply.github.com>
1 participant
This PR updates the Iceberg C++ website structure to follow best practices and adds the missing 0.2.0 release information.
Changes:
Refactored Website Structure:
index.md(which was a Contributing guide) tocontributing.md.index.mdlanding page with an introduction, key features, and quick links.getting-started.mdguide with build and installation instructions extracted from the README.Added 0.2.0 Release:
releases.mdto include Apache Iceberg C++ 0.2.0 (released Jan 26, 2025).Updated Navigation:
mkdocs.ymlto reflect the new page structure (Home, Getting Started, Releases, Contributing, API Docs).Verified the documentation builds correctly using
mkdocs build.PR created automatically by Jules for task 15720229986773954136 started by @wgtmac