Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

vdk-gdp-execution-id: a Generative Data Pack expanding with execution ID #1877

Merged
merged 4 commits into from
Apr 19, 2023

Conversation

ivakoleva
Copy link
Contributor

A quick Generative Data Pack plugin, that detect the Data Job execution ID in runtime, and decorates the data sent for ingestion with a new micro-dimension. Its name is configurable, where the default value is vdk_gdp_execution_id.

Created the plugin, described the README.md examples.

Testing done: added functional tests;
additionally, I tested with a real-world VDK deployment environment, that uses an async ingestion to a relational DB.

A quick Generative Data Pack plugin, that detect the Data Job
execution ID in runtime, and decorates the data sent for ingestion
with a new micro-dimension. Its name is configurable, where the
default value is `vdk_gdp_execution_id`.

Created the plugin, described the README.md examples.

Testing done: added functional tests;
additionally, I tested with a real-world VDK deployment environment,
that uses an async ingestion to a relational DB.

Signed-off-by: ivakoleva <iva.koleva@clearcode.bg>
ivakoleva and others added 2 commits April 11, 2023 10:18
A quick Generative Data Pack plugin, that detect the Data Job
execution ID in runtime, and decorates the data sent for ingestion
with a new micro-dimension. Its name is configurable, where the
default value is `vdk_gdp_execution_id`.

Created the plugin, described the README.md examples.

Testing done: added functional tests;
additionally, I tested with a real-world VDK deployment environment,
that uses an async ingestion to a relational DB.

Signed-off-by: ivakoleva <iva.koleva@clearcode.bg>
Copy link
Collaborator

@murphp15 murphp15 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

@ivakoleva ivakoleva requested a review from doks5 April 19, 2023 07:15
@ivakoleva ivakoleva enabled auto-merge (squash) April 19, 2023 07:32
@ivakoleva ivakoleva merged commit 06b11b3 into main Apr 19, 2023
5 of 6 checks passed
@ivakoleva ivakoleva deleted the person/ikoleva/vdk-gdp-execution-id branch April 19, 2023 07:47
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.

None yet

5 participants