Don't print empty operations

sverrehu commented 2 years ago

Please check if the PR fulfills these requirements
[* ] The commit messages are descriptive
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[ ] An issue has been created for the pull requests. Some issues might require previous discussion.
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

I only changed ExecutionPlan.java and TopicManagerTest.java. The other two files were changed due to the embedded formatter.

purbon commented 2 years ago

❤️ thanks a lot for your contributions @sverrehu! you guys really rock and are super helpful!

purbon commented 2 years ago

re: due to the embedded formatter. I do really have to find a better way, it piss me off often this formatter :-(

sverrehu commented 2 years ago

❤️ thanks a lot for your contributions @sverrehu! you guys really rock and are super helpful!

I think I was the one who introduced those blank lines in the first place. ;-) But thanks!

kafka-ops / julie