Closed ppanopticon closed 6 months ago
I assigned this to you @lucaro since you added the existing retrieval operations.
I'll have a look
Added relevant changes in https://github.com/vitrivr/vitrivr-engine/tree/feature/68. @ppanopticon please have a look to see if that is along the lines of what you had in mind.
@ppanopticon any comments?
Looks good to me. If it works we can merge it. I opened PR #71 .
Done. I did not have the opportunity to extensively test this, but since the changes are only structural and not semantic and it compiles, I don't see much that would have caused a behavioral change.
Task Description
In light of the changes introduced in #51, the design of the retrieval pipeline needs slight adjustment. I did not address this in #51 since I'm not too familiar with how the different operators work and what input they expect.
Roughly, I'd propose the following changes:
org.vitrivr.engine.core.operators.retrieve.Aggregator
andorg.vitrivr.engine.core.operators.retrieve.Transformer
can be replaced by theorg.vitrivr.engine.core.operators.general.Transformer
interface.org.vitrivr.engine.core.operators.general.TransformerFactory
probably needs some adjustment, to support both retrieval and extraction. Easiest way would be to make the methodnewTransformer
acceptContext
as parameter instead ofIndexContext
.Dependencies
Boundary Conditions
None