leesf commented on a change in pull request #2106:
URL: https://github.com/apache/hudi/pull/2106#discussion_r497488441
##########
File path:
hudi-client/src/main/java/org/apache/hudi/client/HoodieWriteClient.java
##########
@@ -186,11 +186,15 @@ protected void rollBackInflightBootstrap() {
* @return JavaRDD[WriteStatus] - RDD of WriteStatus to inspect errors and
counts
*/
public JavaRDD<WriteStatus> upsert(JavaRDD<HoodieRecord<T>> records, final
String instantTime) {
+ return upsert(records, instantTime, null);
+ }
+
+ public JavaRDD<WriteStatus> upsert(JavaRDD<HoodieRecord<T>> records, final
String instantTime, String schema) {
Review comment:
would we use `Option<String>` here to avoid NPE?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
