Skip to content

fix: remove deprecated command/flags W-18384354#1237

Merged
WillieRuemmele merged 3 commits intomainfrom
cd/remove-deprecated-flags
Apr 29, 2025
Merged

fix: remove deprecated command/flags W-18384354#1237
WillieRuemmele merged 3 commits intomainfrom
cd/remove-deprecated-flags

Conversation

@cristiand391
Copy link
Copy Markdown
Member

@cristiand391 cristiand391 commented Apr 28, 2025

What does this PR do?

follow up of #1134

Removes:

  • data query resume command
  • --wait, --bulk and --async flags in data query

What issues does this PR fix or reference?

@W-18384354@

@cristiand391 cristiand391 requested a review from a team as a code owner April 28, 2025 18:04
@cristiand391 cristiand391 added the bug Something isn't working label Apr 28, 2025
@git2gus
Copy link
Copy Markdown

git2gus bot commented Apr 28, 2025

This issue has been linked to a new work item: W-18384354

@cristiand391 cristiand391 changed the title fix: remove deprecated command/flags fix: remove deprecated command/flags W-18384354 Apr 28, 2025
}
}

export class BulkQueryRequestCache extends BulkDataRequestCache {
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

we have 1 class per data-resumable command (so that each has it's own json file). This one was used only by data query/data query resume, data export resume has a separate one.

@WillieRuemmele WillieRuemmele merged commit e4e62d3 into main Apr 29, 2025
33 checks passed
@WillieRuemmele WillieRuemmele deleted the cd/remove-deprecated-flags branch April 29, 2025 18:05
@WillieRuemmele
Copy link
Copy Markdown
Contributor

QA Notes


👋 bye

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants