[FLINK-14871] Better formatter of toString method for StateTransition

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.9.1
• Fix Version/s: 1.10.0
• Component/s: Library / CEP
• Labels: pull-request-available
• Assignee: chaiyongqiang
• Reporter: chaiyongqiang
• Created: 20/Nov/19 07:33
• Updated: 20/Nov/19 14:05
• Resolved: 20/Nov/19 14:05

Description

The toString method in StateTransition does not have a good format. 

@Override
	public String toString() {
		return new StringBuilder()
				.append("StateTransition(")
				.append(action).append(", ")
				.append("from ").append(sourceState.getName())
				.append("to ").append(targetState.getName())
				.append(condition != null ? ", with condition)" : ")")
				.toString();
	}

The problem is there's no separator between `sourceState.getName()` and "to " which leads to a bad format like the attachment showing.

A blank space is more friendly.

@Override
	public String toString() {
		return new StringBuilder()
				.append("StateTransition(")
				.append(action).append(", ")
				.append("from ").append(sourceState.getName())
				.append(" to ").append(targetState.getName())
				.append(condition != null ? ", with condition)" : ")")
				.toString();
	}

Issue Links

• links to: GitHub Pull Request #10261