anishek commented on a change in pull request #965: HIVE-23039 Checkpointing
for repl dump bootstrap phase
URL: https://github.com/apache/hive/pull/965#discussion_r401496558
##########
File path: ql/src/java/org/apache/hadoop/hive/ql/exec/repl/util/ReplUtils.java
##########
@@ -296,4 +300,17 @@ public static boolean includeAcidTableInDump(HiveConf
conf) {
public static boolean tableIncludedInReplScope(ReplScope replScope, String
tableName) {
return ((replScope == null) ||
replScope.tableIncludedInReplScope(tableName));
}
+
+ public static boolean dataCopyCompleted(Path toPath, HiveConf conf) throws
IOException {
+ FileSystem dstFs = null;
+ dstFs = toPath.getFileSystem(conf);
+ if (dstFs.exists(new Path(toPath, ReplUtils.COPY_ACKNOWLEDGEMENT))) {
Review comment:
do return of condition directly rather than using if clause
----------------------------------------------------------------
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]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
