flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ufuk Celebi (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (FLINK-9832) Allow commas in job submission query params
Date Thu, 12 Jul 2018 11:13:00 GMT

    [ https://issues.apache.org/jira/browse/FLINK-9832?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16541477#comment-16541477
] 

Ufuk Celebi commented on FLINK-9832:
------------------------------------

I didn't look into the code, but is it possible to fix the utility that is responsible for
parsing the query parameter in the mean time? In general, it should be possible as it works
in 1.4.


> Allow commas in job submission query params
> -------------------------------------------
>
>                 Key: FLINK-9832
>                 URL: https://issues.apache.org/jira/browse/FLINK-9832
>             Project: Flink
>          Issue Type: Bug
>          Components: REST
>    Affects Versions: 1.5.1
>            Reporter: Ufuk Celebi
>            Priority: Minor
>
> As reported on the user mailing list in the thread "Run programs w/ params including
comma via REST api" [1], submitting a job with mainArgs that include a comma results in an
exception.
> To reproduce submit a job with the following mainArgs:
> {code}
> --servers 10.100.98.9:9092,10.100.98.237:9092
> {code}
> The request fails with
> {code}
> Expected only one value [--servers 10.100.98.9:9092, 10.100.98.237:9092].
> {code}
> As a work around, users have to use a different delimiter such as {{;}}.
> The proper fix of this API would make these params part of the {{POST}} request instead
of relying on query params (as noted in FLINK-9499). I think it's still valuable to fix this
as part of a bug fix release for 1.5.
> [1] http://apache-flink-user-mailing-list-archive.2336050.n4.nabble.com/Run-programs-w-params-including-comma-via-REST-api-td19662.html



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Mime
View raw message