peterphitran commented on PR #39064:
URL: https://github.com/apache/beam/pull/39064#issuecomment-4786288057
@ahmedabu98 @Abacn updated based on the feedback (full details in the PR
description):
Dropped the String.valueOf wrapping, found the issue was a build-infra bug:
on a Java 11 host, the forked Java 17 compile passes javac args via an
@argfile, and the \. in Beam's Checker skipUses regex gets mangled in transit,
silently dropping the Logger nullness suppression. Fixed in BeamModulePlugin by
using [.] instead of \. (same meaning, no backslash to mangle). The IO source
is back to plain LOG.info(...). xlang launch now uses java${ver}Home (via
-PtestJavaVersion) instead of hardcoded java17.
--
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]
