sankarh commented on a change in pull request #579: HIVE-21109 : Support stats replication for ACID tables. URL: https://github.com/apache/hive/pull/579#discussion_r270653325
########## File path: itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/parse/TestStatsReplicationScenarios.java ########## @@ -269,11 +294,23 @@ private String dumpLoadVerify(List<String> tableNames, String lastReplicationId, WarehouseInstance.Tuple dumpTuple = primary.run("use " + primaryDbName) .dump(primaryDbName, lastReplicationId, withClauseList); + // Load, if necessary changing configuration. if (parallelLoad) { replica.hiveConf.setBoolVar(HiveConf.ConfVars.EXECPARALLEL, true); } + // Fail load if for testing failure and retry scenario. Fail the load while setting + // checkpoint for a table in the middle of list of tables. + if (failRetry) { + if (lastReplicationId == null) { + failBootstrapLoad(dumpTuple, tableNames.size()/2); + } else { + failIncrementalLoad(dumpTuple, tableNames.size()/2); Review comment: It is not directly mapped that one event per table. So, this value of tableNames.size()/2 may fail even before applying update state event. If we want to fail the incremental load after a fixed event, then need to get the event count by dumping it after that operation at source. ---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For additional commands, e-mail: gitbox-h...@hive.apache.org