xicm commented on code in PR #7355:
URL: https://github.com/apache/hudi/pull/7355#discussion_r1049451179
##########
hudi-integ-test/src/test/java/org/apache/hudi/integ/ITTestHoodieSanity.java:
##########
@@ -193,19 +193,6 @@ public void testRunHoodieJavaApp(String command, String
hiveTableName, String ta
assertEquals(80,
Integer.parseInt(stdOutErr.getLeft().substring(stdOutErr.getLeft().lastIndexOf("\n")).trim()),
"Expecting 80 rows to be present in the snapshot table");
}
-
- // Make the HDFS dataset non-hoodie and run the same query; Checks for
interoperability with non-hoodie tables
- // Delete Hoodie directory to make it non-hoodie dataset
- executeCommandStringInDocker(ADHOC_1_CONTAINER, "hdfs dfs -rm -r " +
hdfsPath + "/.hoodie", true);
-
Review Comment:
We need meta table when we query hive now, this case doesn't fit anymore.
--
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]
