Skip to content

feat: add support for 'Contains' on a collection in the implementation of IQueryable.Where#54

Merged
PierrePicard20 merged 6 commits intomasterfrom
ppi/queryableContains
Jan 23, 2026
Merged

feat: add support for 'Contains' on a collection in the implementation of IQueryable.Where#54
PierrePicard20 merged 6 commits intomasterfrom
ppi/queryableContains

Conversation

@PierrePicard20
Copy link
Copy Markdown
Contributor

@PierrePicard20 PierrePicard20 commented Jan 21, 2026

Motivation

Use the IQueryable interface to query blobs having a callback defined and being either completed or aborted. This require the support of the Contains method in the Where expression.

Description

Add support for 'Contains' on a collection in the implementation of IQueryable.Where, and use it for callbacks invocation.

Testing

Non regression tests.
New unit tests added on the support of Contains expressions.

Impact

Callbacks invocation.

Additional Information

Added extensions method WithPageSize(int pageSize) for IQueryable instances: set the page size of the answers.
Removal of filtering on blobs's OpaqueId field.

Checklist

  • My code adheres to the coding and style guidelines of the project.
  • I have performed a self-review of my code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • I have thoroughly tested my modifications and added tests when necessary.
  • Tests pass locally and in the CI.
  • I have assessed the performance impact of my modifications.

@PierrePicard20 PierrePicard20 changed the base branch from master to ppi/parallelDownload January 22, 2026 14:27
Base automatically changed from ppi/parallelDownload to master January 23, 2026 09:06
@PierrePicard20 PierrePicard20 marked this pull request as ready for review January 23, 2026 10:14
@PierrePicard20 PierrePicard20 merged commit d31e3ac into master Jan 23, 2026
6 checks passed
@PierrePicard20 PierrePicard20 deleted the ppi/queryableContains branch January 23, 2026 16:13
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