[GitHub] [hudi] xushiyan commented on a diff in pull request #6694: [HUDI-4855] Add missing table configs for bootstrap in Deltastreamer

[GitBox <gi...@apache.org>]

xushiyan commented on code in PR #6694:
URL: https://github.com/apache/hudi/pull/6694#discussion_r993376698


##########
hudi-utilities/src/test/java/org/apache/hudi/utilities/functional/TestHoodieDeltaStreamer.java:
##########
@@ -639,11 +639,14 @@ public void testBulkInsertsAndUpsertsWithBootstrap() throws Exception {
     Dataset<Row> sourceDf = sqlContext.read()
         .format("org.apache.hudi")
         .load(tableBasePath);
-    sourceDf.write().format("parquet").save(bootstrapSourcePath);
+    // TODO(HUDI-4944): fix the test to use a partition column with slashes (`/`) included
+    //  in the value.  Currently it fails the tests due to slash encoding.
+    sourceDf.write().format("parquet").partitionBy("rider").save(bootstrapSourcePath);

Review Comment:
   should we fix it in this PR? trying to understand what's effort here



-- 
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: commits-unsubscribe@hudi.apache.org

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