slachiewicz commented on PR #3820: URL: https://github.com/apache/thrift/pull/3820#issuecomment-5599410257
Both lists check out. Every entry is corrected in #3834, verified against the source at 5c5e93e1e rather than taken on report: - Java SE now reads 8 / 17, on the CI bounds and the `options.release = 8` toolchain you pointed at. - Rust TLS, domain sockets and ThreadPool are marked supported: the `rustls` feature, `TServer::listen_uds` under `#[cfg(unix)]`, and the fixed-size worker pool in `TServer`. - Python Memory and Framed transports, and the Threaded and ThreadPool servers. - Kotlin reads 2.4.10, and Multiplex, zlib, Nonblocking and ThreadPool are marked supported. - Haxe Memory, on `TMemoryStream` and its round-trip tests. Since the table can't carry a condition, that PR also adds a short notes section: the `rustls` feature is off by default, `listen_uds` is Unix-only, and the Java and Kotlin rows report tested bounds rather than a maximum supported release. One I left for a separate discussion: Rust has a single server type, a fixed-size pool, and the row now marks it as both Threaded and ThreadPool. ThreadPool is clearly right; whether Threaded should stay is a judgement call I didn't want to bury in a corrections PR. *This comment was created with AI assistance.* -- 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]
