mengw15 opened a new pull request, #6033:
URL: https://github.com/apache/texera/pull/6033

   ### What changes were proposed in this PR?
   
   `bin/bootstrap-lakekeeper.sh` ran Lakekeeper natively on the host, a 
workaround from when Docker Lakekeeper wasn't reachable from host-native Texera 
(the warehouse's S3 endpoint was registered as a Docker-internal hostname the 
host couldn't resolve, and switching it to `localhost` broke the 
Lakekeeper-container → MinIO link).
   
   `bin/local-dev.sh` (#5961, merged) now runs Lakekeeper in Docker with the S3 
endpoint set to the host's LAN IP — reachable from both the host-native Texera 
and the Lakekeeper container — so the native-bootstrap path is redundant.
   
   This PR removes:
   - `bin/bootstrap-lakekeeper.sh` (the script)
   - `docs/tutorials/guide-to-launch-lakekeeper.md` (its tutorial — every step 
documented the now-removed native flow; nothing in it survives)
   - the `bootstrap-lakekeeper.sh` row in `bin/README.md`
   
   Documenting the `local-dev.sh` Lakekeeper flow, if wanted, is a separate 
follow-up.
   
   ### Any related issues, documentation, discussions?
   
   Closes #6032. Follows up #5961.
   
   ### How was this PR tested?
   
   No code change — this removes a host-only bootstrap script and its 
documentation, neither of which is invoked by CI, the build, containers, or any 
other script (CI provisions Lakekeeper independently via `docker run` / a 
native binary in `build.yml`). `grep -rn bootstrap-lakekeeper` over the repo 
confirms no remaining references after the removal.
   
   ### Was this PR authored or co-authored using generative AI tooling?
   
   Generated-by: Claude Code (claude-opus-4-7)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to