tillrohrmann commented on a change in pull request #6738: [FLINK-10126] There
should be a Scala DataSource
URL: https://github.com/apache/flink/pull/6738#discussion_r224731539
##########
File path: pom.xml
##########
@@ -1613,6 +1613,7 @@ under the License.
<exclude>org.apache.flink.api.scala.hadoop.mapred.HadoopOutputFormat</exclude>
<exclude>org.apache.flink.api.scala.hadoop.mapreduce.HadoopInputFormat</exclude>
<exclude>org.apache.flink.api.scala.hadoop.mapreduce.HadoopOutputFormat</exclude>
+
<exclude>org.apache.flink.api.scala.ExecutionEnvironment</exclude>
Review comment:
Would `org.apache.flink.api.scala.ExecutionEnvironment#createInput` work as
an exclusion? This is then more specific and would not exclude the complete
`ExecutionEnvironment`.
The danger is still that we would not detect any breaking API changes made
to these methods in the future. Could we maybe add a manual test which fails if
this happens and add a note stating why we have these tests?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
