morningman commented on a change in pull request #1323: This commit has brought
contribution to streaming mini load
URL: https://github.com/apache/incubator-doris/pull/1323#discussion_r294587515
##########
File path: fe/src/main/java/org/apache/doris/http/rest/GetLoadInfoAction.java
##########
@@ -63,7 +64,15 @@ public void executeWithoutPassword(AuthorizationInfo
authInfo, BaseRequest reque
if (redirectToMaster(request, response)) {
return;
}
- catalog.getLoadInstance().getJobInfo(info);
+ try {
+ try {
+ catalog.getLoadInstance().getJobInfo(info);
+ } catch (DdlException e) {
+ catalog.getLoadManager().getLoadJobInfo(info);
Review comment:
LoadManager is your new implementation, so only history load job may exist
in LoadInstance.
If you check LoadInstance first, then you have to check both LoadInstance
and LoadManager for every load job info request in the future, which is
inefficient.
----------------------------------------------------------------
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]
