`UpdateTask`  some logic errors

**Describe the bug**
Finally code block will always execute. So `incFailCount()` will always execute. And when `queryInstancesOfService` error delayTime = -1, this task will also be stopped.
Why need add `delayTime > 0` check , should a default value be given.
https://github.com/alibaba/nacos/blob/eb36581c2c4b0a9bfc0a7837452285d0afcf530d/client/src/main/java/com/alibaba/nacos/client/naming/core/ServiceInfoUpdateService.java#L190-L201

**Desktop (please complete the following information):**
 - OS: [e.g. Centos]
 - Version [e.g. nacos-server 2.0.0, nacos-client 2.0.0]
 - Module [e.g. naming]
 - SDK [java]

**Additional context**
Add any other context about the problem here.

	}
	// TODO multiple time can be configured.
	delayTime = serviceObj.getCacheMillis() * DEFAULT_UPDATE_CACHE_TIME_MULTIPLE;
	resetFailCount();
	} catch (Throwable e) {
	NAMING_LOGGER.warn("[NA] failed to update serviceName: " + groupedServiceName, e);
	} finally {
	incFailCount();
	if (delayTime > 0) {
	executor.schedule(this, Math.min(delayTime << failCount, DEFAULT_DELAY * 60), TimeUnit.MILLISECONDS);
	}
	}

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

`UpdateTask` some logic errors #5028

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

UpdateTask some logic errors #5028

Description

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions

`UpdateTask` some logic errors #5028