Skip to content

[fix][fn] Forward source message properties in Python runtime - #26191

Merged
lhotari merged 4 commits into
apache:masterfrom
sbourkeostk:python-forward-properties
Jul 25, 2026
Merged

[fix][fn] Forward source message properties in Python runtime#26191
lhotari merged 4 commits into
apache:masterfrom
sbourkeostk:python-forward-properties

Conversation

@sbourkeostk

@sbourkeostk sbourkeostk commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Fixes #17773

Motivation

The Python functions runtime currently ignores the forward-source-message-property option and does not forward message properties. This PR fixes that.

Modifications

python_instance.py has been modified to check for the flag and copy the message properties if requested. Tests have been added.

Verifying this change

  • Make sure that the change passes the CI checks.

This change added tests and can be verified by running the Python functions tests.

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

  • Dependencies (add or upgrade a dependency)
  • The public API
  • The schema
  • The default values of configurations
  • The threading model
  • The binary protocol
  • The REST endpoints
  • The admin CLI options
  • The metrics
  • Anything that affects deployment

@lhotari lhotari added this to the 5.0.0-M2 milestone Jul 25, 2026
@lhotari
lhotari merged commit f07e0ed into apache:master Jul 25, 2026
42 checks passed
lhotari pushed a commit that referenced this pull request Jul 25, 2026
Co-authored-by: Stephen Bourke <sbourke@beyond.com>
(cherry picked from commit f07e0ed)
lhotari pushed a commit that referenced this pull request Jul 25, 2026
Co-authored-by: Stephen Bourke <sbourke@beyond.com>
(cherry picked from commit f07e0ed)
sandeep-ctds pushed a commit to datastax/pulsar that referenced this pull request Jul 31, 2026
…#26191)

Co-authored-by: Stephen Bourke <sbourke@beyond.com>
(cherry picked from commit f07e0ed)
sandeep-ctds pushed a commit to datastax/pulsar that referenced this pull request Jul 31, 2026
…#26191)

Co-authored-by: Stephen Bourke <sbourke@beyond.com>
(cherry picked from commit f07e0ed)
sandeep-ctds pushed a commit to datastax/pulsar that referenced this pull request Jul 31, 2026
…#26191)

Co-authored-by: Stephen Bourke <sbourke@beyond.com>
(cherry picked from commit f07e0ed)
nodece pushed a commit to ascentstream/pulsar that referenced this pull request Aug 28, 2026
…#26191)

Co-authored-by: Stephen Bourke <sbourke@beyond.com>
(cherry picked from commit f07e0ed)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] python function ignores --forward-source-message-property true

2 participants