Skip to content

Add option to skip input parameter validation#854

Merged
dirkkul merged 3 commits intomainfrom
skip_parameter_validation
Feb 5, 2024
Merged

Add option to skip input parameter validation#854
dirkkul merged 3 commits intomainfrom
skip_parameter_validation

Conversation

@dirkkul
Copy link
Collaborator

@dirkkul dirkkul commented Feb 5, 2024

No description provided.

@dirkkul dirkkul requested a review from a team as a code owner February 5, 2024 06:55
Copy link
Collaborator

@tsmith023 tsmith023 left a comment

Choose a reason for hiding this comment

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

if self._validate_arguments is missing in:

  • collection.with_tenant
  • collections.delete
  • collections.exists
  • collections.list_all
  • data.exists
  • tenants.create
  • tenants.remove
  • tenants.update
  • Methods in aggregations/base.py

It could be that these aren't useful to cancel since they aren't high throughput methods?

@dirkkul dirkkul merged commit 12a2d54 into main Feb 5, 2024
@dirkkul dirkkul deleted the skip_parameter_validation branch February 5, 2024 09:17
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