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

Chesnay Schepler commented on FLINK-12602:
------------------------------------------

Mirroring my comment from the ML; there's no reason to keep that thread going 
now that we have a JIRA.

There are 3 modules that have a suffix, which don't need it:

 * flink-connector-hive

 * flink-queryable-state-client-java

 * flink-table-api-scala

All other modules do need it since they have dependencies with a scala-suffix 
(mostly on runtime and streaming-java).

Your change does make sense to me, but there's likely another issue in the 
preceding logic that determines which module is scala-free.
flink-tests for example should not be considered scala-free, since it relies on 
flink-runtime which contains scala and hence the scala-lang dependencies, but 
it apparently is given that your change detects something. 

> Correct the flink pom `artifactId` config and scala-free check logic
> --------------------------------------------------------------------
>
>                 Key: FLINK-12602
>                 URL: https://issues.apache.org/jira/browse/FLINK-12602
>             Project: Flink
>          Issue Type: Bug
>          Components: Build System
>    Affects Versions: 1.9.0
>            Reporter: sunjincheng
>            Priority: Major
>
> I find a shell issue in `verify_scala_suffixes.sh`(line 145) as follows:
> {code}
> grep "${module}_\d\+\.\d\+</artifactId>" "{}"
> {code}
> This code want to find out all modules that the module's `artifactId`  with a 
> `scala_binary_version` suffix. 
> but the problem is our all `artifactId` value is in the pattern of 
> `XXX_${scala.binary.version}`, such as:
> {code}
> <artifactId>flink-tests_${scala.binary.version}</artifactId>
> {code}
> then the result always empty, so this check did not take effect.
> I have already initiated a discussion of the issue. Please check the Mail 
> thread here for details.
> http://mail-archives.apache.org/mod_mbox/flink-dev/201905.mbox/%3ccajsjtkw+8mcsc0fvneyaovl_ttrr_uuosx-tfgxj5gfqp1a...@mail.gmail.com%3E



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

Reply via email to