Nested Documents

Question

Nested Documents

Opened this issue 8 years ago · 12 comments

Mortagus commented 8 years ago

Hello everyone,

I'm working now on a proof of concept about Solr documents with nested data.

I'm wondering if there is a implementation of the "childDocuments" from Solr server.

Answer 1 · 2017-03-21T15:00:34.000Z

The solarium client supports nested documents but is not yet implemented in this bundle

Answer 2 · 2017-03-21T15:29:19.000Z

ok thanks for your help :-)

Answer 3 · 2017-03-21T15:47:53.000Z

Would it be possible to update the composer.json file ? The solarium library is at the 3.8 version and apparently it's needed to have the nested document feature (if I read correctly the documentation)

Answer 4 · 2017-03-21T15:50:51.000Z

sure! where is your pull-request? ;)

Answer 5 · 2017-03-24T13:20:26.000Z

Can you develop what you mean ?

I've never made any contribution into a github repo before.

How do I do that pull request ?

Answer 6 · 2017-03-24T13:25:41.000Z

After some reflexion, I think I know what it is. But I've never done that on github.
Can you give me the procedure, just to be sure I'm not doing something wrong ;-)

Answer 7 · 2017-03-24T13:28:07.000Z

if you want a higher version of the solarium client just run: composer require solarium/solarium:^3.8

Answer 8 · 2017-03-24T13:33:59.000Z

This is what I wanted to do, but I discussed this with my coworker, and we decided it was a secondary requirement we didn't want in our composer file. That dependency is linked to your project, so it has to stay there.

It's just about compartmentalizing the dependencies.

Answer 9 · 2017-03-24T15:25:08.000Z

Well, my bundle requires version 3.6 and you need version 3.8, so we speak here about a project-specific dependency.

The next release contains a nested-document feature which requires the version 3.8. If you can wait then you do need to update your project-dependencies.

Answer 10 · 2017-03-27T08:40:17.000Z

Hello Florian,

thanks for your answer.
I think you're right. This need is specific to our project so I'm going to update the dependency in our own composer file ;-).

Can I ask your opinion: do you think there will be side effects if we're using the 3.8 version ?

Answer 11 · 2017-03-27T08:53:14.000Z

version 3.8 does not contain critical changes and I have not notice strange behavior with my local setup. So everything should work well.

Answer 12 · 2017-06-14T11:24:15.000Z

Hello @floriansemm ,

I'm still on this feature ^^ (a lot of work has been done on other things though).
Now I need to store a list of element as array.
But, like you said in a previous comment: I would need to implement childDocuments.

Can you give me some advice to implement something that would respect good practice ?
Do you think it would be interesting to extend you Field class ?