ferenc-csaky commented on code in PR #57:
URL:
https://github.com/apache/flink-connector-mongodb/pull/57#discussion_r2741099469
##########
flink-connector-mongodb/pom.xml:
##########
@@ -73,6 +73,13 @@ under the License.
</dependency>
<!-- Table ecosystem -->
+ <!-- Table API integration tests -->
+ <dependency>
+ <groupId>org.apache.flink</groupId>
+ <artifactId>flink-table-planner-loader</artifactId>
+ <version>${flink.version}</version>
+ <scope>test</scope>
+ </dependency>
Review Comment:
nit: I'd put this to L125, where the old planner dep got removed. I do not
see the reason why move this up, the table related test deps are already
grouped there, this breaks that pattern.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
