[GitHub] [flink] nateab commented on a diff in pull request #22359: [FLINK-31660][table] add jayway json path dependency to table-planner

["nateab (via GitHub)" <gi...@apache.org>]

nateab commented on code in PR #22359:
URL: https://github.com/apache/flink/pull/22359#discussion_r1160388987


##########
flink-table/flink-table-planner/pom.xml:
##########
@@ -301,6 +301,13 @@ under the License.
 			<version>${project.version}</version>
 			<scope>test</scope>
 		</dependency>
+
+		<dependency>
+			<groupId>com.jayway.jsonpath</groupId>
+			<artifactId>json-path</artifactId>
+			<version>${jsonpath.version}</version>
+			<scope>test</scope>

Review Comment:
   Okay gotcha, i've removed the test scope. However, i am no longer able to get the `KafkaChangelogTableITCase` test running in the IDE anymore, even after rebuilding the modules on the command line. I have tried adding it individually and in various combinations in `flink-table-planner-loader-bundle, flink-table-planner-loader, flink-table-planner_${scala.binary.version}`. Did you have to do anything else to get it to work?



-- 
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: issues-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org