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

[AP-2038] implementing reset_state command #1192

Merged
merged 5 commits into from
Jan 9, 2025

Conversation

amofakhar
Copy link
Contributor

@amofakhar amofakhar commented Jan 8, 2025

Context

Added the new command reset_state to reset state file for log-based tables in PostgresSQL taps.

Types of changes

What types of changes does your code introduce to PipelineWise?
Put an x in the boxes that apply

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update (if none of the other choices apply)

Checklist

  • I have read the CONTRIBUTING doc
  • Description above provides context of the change
  • I have added tests that prove my fix is effective or that my feature works
  • Unit tests for changes (not needed for documentation changes)
  • CI checks pass with my changes
  • Branch name starts with AP-NNN (if applicable. AP-NNN = JIRA ID)
  • Relevant documentation is updated including usage instructions

Details from ticket: AP-2038

PPW :: Command option for reseting State file

h2. Context

Please give context of the task, the problem that it solves, how impactful it is ..etc

h2. Suggested Implementation(s)

Any ideas on solutions or implementation

@amofakhar amofakhar requested a review from a team as a code owner January 8, 2025 09:27
@wise-github-bot-app
Copy link

Reminder: Classification of Untagged Assets

This is currently a non-blocking check, and it will not prevent your Pull Request from being merged.

The CODEOWNER analytics-platform of this repository has some untagged assets that need classification:

Please review and classify these assets when you approve the code. Feel free to merge your PR as planned.

For any questions or assistance, join us on #untagged-assets-check-support. You can find more information about data ownership and how to video in confluence at Data Ownership / How to classify data system?.

Thank you for your cooperation! This will help us retain our license and achieve our Mission of money without borders!

1 similar comment
@wise-github-bot-app
Copy link

Reminder: Classification of Untagged Assets

This is currently a non-blocking check, and it will not prevent your Pull Request from being merged.

The CODEOWNER analytics-platform of this repository has some untagged assets that need classification:

Please review and classify these assets when you approve the code. Feel free to merge your PR as planned.

For any questions or assistance, join us on #untagged-assets-check-support. You can find more information about data ownership and how to video in confluence at Data Ownership / How to classify data system?.

Thank you for your cooperation! This will help us retain our license and achieve our Mission of money without borders!

louis-pie
louis-pie previously approved these changes Jan 8, 2025
@amofakhar amofakhar merged commit 687ff6b into master Jan 9, 2025
13 of 14 checks passed
@amofakhar amofakhar deleted the AP-2038_reset_state_option branch January 9, 2025 09:19
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