[SENTRY-1676] FullUpdateInitializer#createInitialUpdate should not throw RuntimeException

• Type: Sub-task
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.0.0
• Fix Version/s: 2.0.0
• Component/s: Hdfs Plugin
• Assignee: Alex Kolbasov
• Reporter: Lei (Eddy) Xu
• Created: 23/Mar/17 18:23
• Updated: 24/Jul/17 15:36
• Resolved: 05/Apr/17 17:56

Description

In FullUpdateInitialize#createInitialUpdate, it wraps any exception occurred in db tasks using RuntimeException

FullUpdateInitialize.createInitialUpdate()

 // Fail the HMS startup if tasks are not all successful and
 // fail on partial updates flag is set in the config.
 if (!callResult.getSuccessStatus() && failOnRetry) {
    throw new RuntimeException(callResult.getFailure());
 }

We should not use RumtimeException here, because it is an unchecked exception, which is easier to miss to catch from the callers.

Issue Links

• relates to: SENTRY-1630 out of sequence error in HMSFollower