@vvcephei Yeah, I could see now what you are saying. You are mostly right. I just want to confirm one thing. You mentioned that we need to change all these fields to Map, and that this is a one time expense. This is true if TaskMetadata is upgradeable i.e. we are adding the Map field in the first place because we want to leave TaskId untouched. So then in the future, if we want to add more metadata, could TaskMetadata be upgraded or should it be left alone? If we need to create a new class every time we make an upgrade, than it kind of defeats the purpose of doing it your way. I have no other problems with your plan. Let me know on what you think. :)
[ Full content available at: https://github.com/apache/kafka/pull/5012 ] This message was relayed via gitbox.apache.org for [email protected]
