aasha commented on a change in pull request #1367:
URL: https://github.com/apache/hive/pull/1367#discussion_r468301322
##########
File path:
itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/parse/TestReplicationScenariosAcrossInstances.java
##########
@@ -1639,12 +1655,68 @@ public void
testFailureUnsupportedAuthorizerReplication() throws Throwable {
.run("insert into table1 values (2)");
try {
primary.dump(primaryDbName, clause);
+ Assert.fail();
} catch (SemanticException e) {
assertEquals("Invalid config error : Authorizer sentry not supported for
replication " +
"for ranger service.", e.getMessage());
assertEquals(ErrorMsg.REPL_INVALID_CONFIG_FOR_SERVICE.getErrorCode(),
ErrorMsg.getErrorMsg(e.getMessage()).getErrorCode());
}
+ //This is now non recoverable error
+ try {
+ primary.dump(primaryDbName, clause);
+ Assert.fail();
+ } catch (Exception e) {
+
assertEquals(ErrorMsg.REPL_FAILED_WITH_NON_RECOVERABLE_ERROR.getErrorCode(),
+ ErrorMsg.getErrorMsg(e.getMessage()).getErrorCode());
+ }
+ //Delete non recoverable marker to fix this
Review comment:
Also load will not fail due to non recoverable error as the dump path
itself is not there. At Replication Semantic analyzer level itself it will just
return.
----------------------------------------------------------------
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
