1. Forum
  2. Usenet
  3. LINUX.DEBIAN.MAINT.JAVA

  • Re: reply: Bazel Removal from Debian

    From tony mancill@21:1/5 to sun min on Wed Feb 22 06:00:01 2023
    On Tue, Feb 21, 2023 at 03:15:18PM +0000, sun min wrote:
    If the test passes regularly, and just sometimes fails under the same conditions, I would consider disabling it in autopkgtest rather than have it not
    make it into bookworm. The perfect is the enemy of the good.

    The autopkgtest always fails .

    The maintainer added a “-XX:+UseParallelGC” for garbge collectors param(target jdk16+) which can boost build performance for about 30% compared with the default GC param “-XX:+UseG1GC”[1].

    In fact all the CI stages can be passed if supplied with the default GC param.

    I suspect If the “+UseParallelGC” should be used with some other jvm params together.But I have no idea.

    Hi sun min,

    It sounds like premature optimization to configure or tune the GC
    algorithm at all. The JVM (any version) will be able to configure
    itself, even if perhaps not optimally, without setting either
    "+UseParallelGC" or "+UseG1GC". Is that all we need to change?

    We should be able to build and ship the package with conservative
    (compatible) defaults and document tuning for better performance in a
    wiki or similar.

    Cheers,
    tony

    -----BEGIN PGP SIGNATURE-----

    iQIzBAABCgAdFiEE5Qr9Va3SequXFjqLIdIFiZdLPpYFAmP1oD4ACgkQIdIFiZdL PpaBxA/+MLLJDLvDDsbZ4O7nnqnISmKoc3yoWkeObe1zJFBFk0LE7wgcannzMIos t+RXHgEN9Qn/Bo1tElTh3R6Yd6qzkY/Oc34KFDAaXL3zMlA5wXxNYTjw1aHw6pXY htsahWZ5PPc0UWTt9iW2MZkKZ+tH2x7gVKhjXZL2o6ZS4bk3vnuFzV2Txi5nWZLk /NFkkrcAL8bIlp5WEgATh8UieAba8Z5/mIFR1Qz+fY/NKyf81CKkpChsy+Z0LOIJ vc/6ghulLS2oy+0dRTROpBw+XPmgfssCRPzWLPfOivzBJvWP7z4rZ4QKgvYxrstm ESnmeufc8Xe5gC8Fvs9EE6vBsQTpFsUpzYXX/7l1l+omxaH4hEWjEbJh15P7ac/T DW/hfgpJbfHB14ytaqgJwLaI72gzGOzVtC88zJlRiH0bCMIg0SlAOk2z0ZEe2775 L7JU9CaT3u9QBYhh4VRh6LTAms7vZc1ubLOgaLdP7kjaBIGxXxuuVzGB0vTYXyU3 KM2uwfWTdyD8GGYmUx1YnCbaoL8vPZl7xOz115slI0/TDFWWC/IyskCgQPJOWgtW cEJ5RWTTnkAxy2ZzS07rgnGMZltm0neZkdzusod034D9sUeHFmZI8ZdRvlh4SjkO Tsx32616jvWEg8e/UPpuM2eeNJo1oSlWMbEJyb9XlWVlJjw7WbE=
    =XZJm
    -----END PGP SIGNATURE-----

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Olek Wojnar@21:1/5 to sun min on Wed Feb 22 23:10:01 2023
    Hi sun min,

    On 2/21/23 10:15, sun min wrote:
    The maintainer added a “-XX:+UseParallelGC” for garbge collectors param(target jdk16+) which can boost build performance for about 30%
    compared with the default GC param “-XX:+UseG1GC”[1].

    I added that in an attempt to override the incorrect flag, but instead
    they are both emitted which still causes the error.

    Besides, if it’s possible to downgrade jdk target version to be more compatible with those GC features.

    That's a great idea! Unfortunately, it looks like JDK 17 is the only one available in bookworm and it is no longer compatible with
    'UseParallelOldGC' so I don't think that we really have any alternatives...

    -Olek

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Thorsten Glaser@21:1/5 to sun min on Thu Feb 23 12:50:01 2023
    On Thu, 23 Feb 2023, sun min wrote:

    It seems that the build system's local jdk overrides those jvm
    parameters we defined in bazelrc or bashsrcript.

    How about asking Doko to locally add that parameter to 17 as a
    Debian-specific patch, just ignoring it?

    (And, ideally, warning.)

    bye,
    //mirabilos
    --
    Infrastrukturexperte • tarent solutions GmbH
    Am Dickobskreuz 10, D-53121 Bonn • http://www.tarent.de/
    Telephon +49 228 54881-393 • Fax: +49 228 54881-235
    HRB AG Bonn 5168 • USt-ID (VAT): DE122264941
    Geschäftsführer: Dr. Stefan Barth, Kai Ebenrett, Boris Esser, Alexander Steeg

    **************************************************** /⁀\ The UTF-8 Ribbon
    ╲ ╱ Campaign against Mit dem tarent-Newsletter nichts mehr verpassen:  ╳  HTML eMail! Also, https://www.tarent.de/newsletter
    ╱ ╲ header encryption!
    ****************************************************

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Emmanuel Bourg@21:1/5 to All on Thu Feb 23 13:20:01 2023
    Le 2023-02-23 12:48, Thorsten Glaser a écrit :

    How about asking Doko to locally add that parameter to 17 as a Debian-specific patch, just ignoring it?

    Patching the JDK is overkill, it's just a flag to change in a script.

    The question is, where is this script?

    Emmanuel Bourg

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Olek Wojnar@21:1/5 to All on Sun Mar 5 17:00:02 2023
    Hello and thanks to everyone who offered advice and suggestions on this!

    Using your suggestions and insights, I dug into the issue with some new ideas. I came to two conclusions:

    1) There does not appear to be anything fundamentally broken with the bazel-bootstrap package...however...

    2) The Java test problems are not trivial and not easily solved at this point in the release cycle. To a large extent, that's because this is a pretty old version of Bazel. We just barely ran out of time to upgrade to 5.0 before the freeze. I think that
    would fix this issue, and others.

    The good news is that all Java dependencies of 5.0 were successfully updated in bookworm so my plan continues to be a backport of 5.0 into bookworm once that package looks good in unstable (after bookworm release).

    Thanks again, everyone!

    -Olek

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)