Y-Wakuta commented on PR #4281:
URL: https://github.com/apache/polaris/pull/4281#issuecomment-4393860481

   Thanks @snazy — agreed on all framing points. A note up front: 
`runtime/server-mysql/` was already removed earlier in this PR (per @jbonofre), 
so any `runtime/server-mysql/...` reference was stale. The driver is now opt-in 
via `-PincludeMysqlDriver=true` on `runtime/server`.
   
   Changes pushed:
   
   1. **CHANGELOG.md** — rewrote along your suggested wording: "source-level 
support", official artifacts don't include the GPL driver, point to 
`runtime/server/README.md` in the unpacked source release.
   2. **`site/content/.../relational-jdbc.md`** — removed detailed 
build/properties/bootstrap from the versioned end-user docs; left a single 
paragraph noting it requires a custom downstream build and pointing to the 
source-tree README. Closing paragraph reworded to "Official artifacts
   support PostgreSQL and H2; MySQL via custom source build only".
   3. **`runtime/server/README.md`** — added a "Building a MySQL-capable server 
(custom downstream build)" section with the full build/config/bootstrap 
details, in the source-tree/developer context.
   4. No raw GitHub links remain in the versioned docs or CHANGELOG.
   
   Let me know if any framing is still off, or if you'd prefer the source-tree 
README to live elsewhere (e.g. a separate `BUILDING_MYSQL.md`).


-- 
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