eventmesh icon indicating copy to clipboard operation
eventmesh copied to clipboard

Published 20 hours ago • verified publisher icon apache

[ISSUE#3834] Repeat the operation using method instead.

Open nikam14 opened this issue 2 years ago • 5 comments
trafficstars

Fixes #3834 .

Motivation

Optimisation.

Modifications

made a method . removed repeated function call. used method .

Documentation

  • Does this pull request introduce a new feature? (no)

nikam14 avatar May 11 '23 06:05 nikam14

image please check ckeckstyle.

Alonexc avatar May 29 '23 07:05 Alonexc

@nikam14 please fix the ci check error.

xwm1992 avatar May 29 '23 09:05 xwm1992

sorry, I can't fix it right now may be later

nikam14 avatar May 30 '23 07:05 nikam14

@nikam14 Are you still working on this issue?

harshithasudhakar avatar Jan 09 '24 17:01 harshithasudhakar

Hi @nikam14 , the issue has been closed because ternary operations existed in EventMesh are few now and there's no need for an addtional utility method. Do you have any additional information to share? Feel free to share your latest insights with us. If you feel that your issue has been resolved, you are welcome to close it. We sincerely appreciate your contributions to Apache EventMesh.

Pil0tXia avatar Jan 24 '24 12:01 Pil0tXia