Skip to content

Conversation

cstamas
Copy link
Member

@cstamas cstamas commented Apr 4, 2025

Ability to "pipe" connectors one onto another in controlled and configured fashion.

The ide is following:

  • first (actually connecting to remote) connector is chosen as today (based on priority)
  • next, new type of factories PipelinedRepositoryConnectorFactory is ordered (by priority) and they can (but does not have to) wrap the delegate in configured order.

Factored out existing RRF connector into new factory, and introduced another OfflinePRCF, as so far, "resolver offline" was in fact managed at different spots, and user was still able to circumvent offline setting. This now connector now wraps connector and refuses going remote if session if offline.

Vanilla resolver does this:
offline( rrf( basic( repo ) ) )

Ultimate goal, envision this:
offline( mimir( sigcheck( rrf( basic( repo ) ) ) ) )


https://issues.apache.org/jira/browse/MRESOLVER-685

Ability to "pipe" connectors one onto another in controlled
and configured fashion.

The ide is following:
- first (actually connecting to remote) connector is chosen as
  today (based on priority)
- next, new type of factories PipelinedRepositoryConnectorFactory
  is ordered (by priority) and they can (but does not have to)
  wrap the delegate in configured order.

Factored out existing RRF connector into new factory, and introduced
another OfflinePRCF, as so far, "resolver offline" was in fact
managed at different spots, and user was still able to circumvent
offline setting. This now connector now wraps connector and
refuses going remote if session if offline.

---

https://issues.apache.org/jira/browse/MRESOLVER-685
@cstamas cstamas self-assigned this Apr 4, 2025
@cstamas cstamas marked this pull request as ready for review April 4, 2025 13:21
@cstamas cstamas added this to the 2.0.8 milestone Apr 4, 2025
@cstamas cstamas merged commit e327728 into apache:master Apr 8, 2025
8 checks passed
@cstamas cstamas deleted the MRESOLVER-685 branch April 8, 2025 12:36
@jira-importer
Copy link

Resolve #1361

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