Skip to content

Conversation

hiltontj
Copy link
Contributor

@hiltontj hiltontj commented Jul 11, 2025

Summary

This is a back-port of the pull request https://github.com/influxdata/influxdb_pro/pull/1011

Add a new system table that allows users to inspect the arguments configured for processing engine triggers. The table has three columns:

  • trigger_name: name of the trigger
  • argument_key: key of the argument
  • argument_value: value of the argument

Each trigger argument appears as a separate row in the table, making it easy to query specific triggers or arguments.

Test plan

  • Code compiles successfully
  • Unit tests added for the new functionality
  • Tests cover empty triggers, triggers without arguments, and triggers with arguments
  • Tests cover multiple triggers with different argument configurations

@hiltontj hiltontj force-pushed the feature/processing-engine-trigger-arguments branch from cefaa81 to 181a107 Compare July 11, 2025 17:42
Add a new system table that allows users to inspect the arguments
configured for processing engine triggers. The table has three columns:
- trigger_name: name of the trigger
- argument_key: key of the argument
- argument_value: value of the argument

Each trigger argument appears as a separate row in the table, making
it easy to query specific triggers or arguments.

Update snapshot files to include processing_engine_trigger_arguments table

Update test snapshots to include the new processing_engine_trigger_arguments
system table in:
- Table listing outputs
- Error messages showing valid table names
- Table summaries

This ensures tests properly reflect the new system table in their
expected outputs.
@hiltontj hiltontj force-pushed the feature/processing-engine-trigger-arguments branch from 181a107 to eed59fe Compare July 11, 2025 18:02
Copy link
Contributor

@praveen-influx praveen-influx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good 🚢

@hiltontj hiltontj merged commit ee2dc26 into main Jul 11, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants