Skip to content

meta data changes#12

Open
narenderqlik wants to merge 2 commits intomasterfrom
SAC-28871-add-new-metadata
Open

meta data changes#12
narenderqlik wants to merge 2 commits intomasterfrom
SAC-28871-add-new-metadata

Conversation

@narenderqlik
Copy link
Copy Markdown

@narenderqlik narenderqlik commented Oct 10, 2025

Description of change

This PR adds parent stream metadata tracking to the Pepperjam tap by introducing a new parent_stream field to stream configurations and updating metadata generation to include parent-tap-stream-id information.

Ticket : SAC-28871

  • Added parent_stream field to stream configurations in the flattened streams structure
  • Updated metadata generation to include parent-tap-stream-id when a parent stream exists

Changes

File Description
tap_pepperjam/streams.py Added parent_stream field to both parent and child stream configurations
tap_pepperjam/schema.py Updated metadata generation to include parent-tap-stream-id for child streams

Manual QA steps

You need to comment out the authentication (auth) part in a discovery function, and then generate a log file to verify the changes.

AI generated code

https://internal.qlik.dev/general/ways-of-working/code-reviews/#guidelines-for-ai-generated-code

  • this PR has been written with the help of GitHub Copilot or another generative AI tool

@narenderqlik narenderqlik requested a review from Copilot October 10, 2025 09:38
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds parent stream metadata tracking to the Pepperjam tap by introducing a new parent_stream field to stream configurations and updating metadata generation to include parent-tap-stream-id information.

  • Added parent_stream field to stream configurations in the flattened streams structure
  • Updated metadata generation to include parent-tap-stream-id when a parent stream exists

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
tap_pepperjam/streams.py Added parent_stream field to both parent and child stream configurations
tap_pepperjam/schema.py Updated metadata generation to include parent-tap-stream-id for child streams

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

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.

3 participants