You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@beam.apache.org by "ASF GitHub Bot (Jira)" <ji...@apache.org> on 2022/05/02 18:56:00 UTC

[jira] [Work logged] (BEAM-13468) Dataflow runner - be more lenient with agent java version

     [ https://issues.apache.org/jira/browse/BEAM-13468?focusedWorklogId=765102&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-765102 ]

ASF GitHub Bot logged work on BEAM-13468:
-----------------------------------------

                Author: ASF GitHub Bot
            Created on: 02/May/22 18:55
            Start Date: 02/May/22 18:55
    Worklog Time Spent: 10m 
      Work Description: guillaumecle commented on code in PR #17274:
URL: https://github.com/apache/beam/pull/17274#discussion_r862224385


##########
runners/core-construction-java/src/main/java/org/apache/beam/runners/core/construction/Environments.java:
##########
@@ -117,8 +121,12 @@ public static JavaVersion forSpecification(String specification) {
           return ver;
         }
       }
-      throw new UnsupportedOperationException(
-          String.format("unsupported Java version: %s", specification));
+      JavaVersion fallback = java11;

Review Comment:
   I added a strict method `forSpecificationStrict`.
   
   the java version ends up being used in
   * `JAVA_SDK_HARNESS_CONTAINER_URL` for which I want to use the lenient version for now
   * `DataflowRunner.fromOptions` for which I want to use the lenient version
   * `DataflowRunner.getDefaultContainerImageNameForJob` for which I want to use the lenient version





Issue Time Tracking
-------------------

    Worklog Id:     (was: 765102)
    Time Spent: 2h 20m  (was: 2h 10m)

> Dataflow runner - be more lenient with agent java version
> ---------------------------------------------------------
>
>                 Key: BEAM-13468
>                 URL: https://issues.apache.org/jira/browse/BEAM-13468
>             Project: Beam
>          Issue Type: New Feature
>          Components: runner-dataflow
>            Reporter: Clément Guillaume
>            Assignee: Clément Guillaume
>            Priority: P2
>         Attachments: BEAM-13468-version-as-option.patch, Screenshot from 2022-01-21 16-10-20.png
>
>          Time Spent: 2h 20m
>  Remaining Estimate: 0h
>
> DataflowRunner currently uses {{Environments.getJavaVersion}} to detect java version and build the Dataflow user agent string.
> {{Environments.getJavaVersion}} only accept a small subset of java version (8, 11 and 17)
> It would be nice the user agent string to accept more exotic java versions to allow running the agent on more exotic jvm (still keeping the beam model itself in java 11).



--
This message was sent by Atlassian Jira
(v8.20.7#820007)