jerrypeng opened a new pull request #9945: URL: https://github.com/apache/pulsar/pull/9945
…(#9907)" This reverts commit d0249e5695c75d8d65a4180fd599b17971368f33. ### Motivation We don't need to add additional configuration for message properties to be set for Pulsar messages produced by a source. Why do we need to add a flag for the source? If the `getProperties()` of the Record interface returns a value, it should just automatically be set as a message property. What is the use case to disable something like this? IMO the behavior should always be forwardSourceMessageProperty always be set to set if it is a source. We should not add useless configurations that has no real value to end users. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
