Code review comment: Is it related with this PR?

Code before review:
private void handleShowRoutineLoad() throws AnalysisException {
ShowRoutineLoadStmt showRoutineLoadStmt = (ShowRoutineLoadStmt) stmt;
List<List<String>> rows = Lists.newArrayList();
List<RoutineLoadJob> routineLoadJobList;
try {
routineLoadJobList = GlobalStateMgr.getCurrentState().getRoutineLoadManager()
.getJob(showRoutineLoadStmt.getDbFullName(),
showRoutineLoadStmt.getName(),
showRoutineLoadStmt.isIncludeHistory());
} catch (MetaNotFoundException e) {
LOG.warn(e.getMessage(), e);
throw new AnalysisException(e.getMessage());
}
if (connectContext.getGlobalStateMgr().isUsingNewPrivilege()) {
Iterator<RoutineLoadJob> iterator = routineLoadJobList.iterator();
while (iterator.hasNext()) {
RoutineLoadJob routineLoadJob = iterator.next();
try {
if (!PrivilegeManager.checkAnyActionOnTable(connectContext,
routineLoadJob.getDbFullName(),
routineLoadJob.getTableName())) {
iterator.remove();
}
} catch (MetaNotFoundException e) {
}
}
}
if (routineLoadJobList != null) {
RoutineLoadFunctionalExprProvider fProvider = showRoutineLoadStmt.getFunctionalExprProvider(this.connectContext);
rows = routineLoadJobList.parallelStream()
.filter(fProvider.getPredicateChain())
.sorted(fProvider.getOrderComparator())
.skip(fProvider.getSkipCount())
.limit(fProvider.getLimitCount())
.map(job -> job.getShowInfo())
.collect(Collectors.toList());
}
if (!Strings.isNullOrEmpty(showRoutineLoadStmt.getName()) && rows.isEmpty()) {
throw new AnalysisException("There is no running job named " + showRoutineLoadStmt.getName()
+ " in db " + showRoutineLoadStmt.getDbFullName()
+ ". Include history? " + showRoutineLoadStmt.isIncludeHistory()
+ ", you can try `show all routine load job for job_name` if you want to list stopped and cancelled jobs");
}
resultSet = new ShowResultSet(showRoutineLoadStmt.getMetaData(), rows);
}

Line under review: throw new AnalysisException("There is no running job named " + showRoutineLoadStmt.getName()

Code after addressing the review:
private void handleShowRoutineLoad() throws AnalysisException {
ShowRoutineLoadStmt showRoutineLoadStmt = (ShowRoutineLoadStmt) stmt;
List<List<String>> rows = Lists.newArrayList();
List<RoutineLoadJob> routineLoadJobList;
try {
routineLoadJobList = GlobalStateMgr.getCurrentState().getRoutineLoadManager()
.getJob(showRoutineLoadStmt.getDbFullName(),
showRoutineLoadStmt.getName(),
showRoutineLoadStmt.isIncludeHistory());
} catch (MetaNotFoundException e) {
LOG.warn(e.getMessage(), e);
throw new AnalysisException(e.getMessage());
}
if (connectContext.getGlobalStateMgr().isUsingNewPrivilege()) {
Iterator<RoutineLoadJob> iterator = routineLoadJobList.iterator();
while (iterator.hasNext()) {
RoutineLoadJob routineLoadJob = iterator.next();
try {
if (!PrivilegeManager.checkAnyActionOnTable(connectContext,
routineLoadJob.getDbFullName(),
routineLoadJob.getTableName())) {
iterator.remove();
}
} catch (MetaNotFoundException e) {
}
}
}
if (routineLoadJobList != null) {
RoutineLoadFunctionalExprProvider fProvider = showRoutineLoadStmt.getFunctionalExprProvider(this.connectContext);
rows = routineLoadJobList.parallelStream()
.filter(fProvider.getPredicateChain())
.sorted(fProvider.getOrderComparator())
.skip(fProvider.getSkipCount())
.limit(fProvider.getLimitCount())
.map(job -> job.getShowInfo())
.collect(Collectors.toList());
}
if (!Strings.isNullOrEmpty(showRoutineLoadStmt.getName()) && rows.isEmpty()) {
throw new AnalysisException("There is no running job named " + showRoutineLoadStmt.getName()
+ " in db " + showRoutineLoadStmt.getDbFullName()
+ ". Include history? " + showRoutineLoadStmt.isIncludeHistory()
+ ", you can try `show all routine load job for job_name` if you want to list stopped and cancelled jobs");
}
resultSet = new ShowResultSet(showRoutineLoadStmt.getMetaData(), rows);
}