[CORE] Update AllocationListener usage after successfully releasing memory

[wForget]

What changes were proposed in this pull request?

We should update AllocationListener usage after successfully releasing memory to avoid exceeding the usage limit.

How was this patch tested?

code refactor

[github-actions]

Thanks for opening a pull request!

Could you open an issue for this pull request on Github Issues?

https://github.com/apache/incubator-gluten/issues

Then could you also rename commit message and pull request title in the following format?

[GLUTEN-${ISSUES_ID}][COMPONENT]feat/fix: ${detailed message}

See also:

• Other pull requests

[wForget]

@zhztheplayer This is just a minor improvement I found while reading the relevant codes. Could you please take a quick look?

[zhztheplayer]

Can we use two different code branches for diff > 0 and diff < 0 ? E.g.,

Suggested change

	int64_t diff = newSize - size;
	updateUsage(diff);
	if (diff > 0) {
	updateUsage(diff);
	}
	bool succeed = delegated_->reallocate(p, size, newSize, out);
	if (!succeed) {
	if (succeed && diff < 0) {
	updateUsage(diff);
	} else if (!succeed && diff > 0) {
	updateUsage(-diff);
	}
	return succeed;
	int64_t diff = newSize - size;
	if (diff >= 0) {
	updateUsage(diff);
	bool succeeded = delegated_->reallocate(p, size, newSize, out);
	if (!succeeded) {
	updateUsage(-diff);
	}
	return succeeded;
	}
	// Code path for diff < 0.
	bool succeeded = delegated_->reallocate(p, size, newSize, out);
	if (succeeded) {
	updateUsage(diff);
	}
	return succeed;

[zhztheplayer]

Just one minor comment. Thanks!