Resolve "Need to restart server to access created ldp source"

Closes #73 (closed)

added 1 deleted label

added 1 commit

• a67f2ed6 - update: reload source on new source

Compare with previous version

By Jean-Baptiste on 2019-08-03T14:46:20 (imported from GitLab project)

unmarked as a Work In Progress

By Jean-Baptiste on 2019-08-03T14:46:53 (imported from GitLab project)

enabled an automatic merge when the pipeline for a67f2ed6 succeeds

By Jean-Baptiste on 2019-08-03T14:46:57 (imported from GitLab project)

mentioned in commit cd9a45ee

By Jean-Baptiste on 2019-08-03T14:47:46 (imported from GitLab project)

merged

By Jean-Baptiste on 2019-08-03T14:47:46 (imported from GitLab project)

I have the feeling this is the wrong way of solving it. I think we should replace all these urls by just one of the form /sources/(?P<source>\w+)/. This wouldn't need any reinstanciation.

What do you think?

We could but it implies to change the model of LDPSource to have one LDPSource per 'type' (with a list of urls) instead of a list of sources merged by 'type' into a container at the client request.

So, since it works now, my advice is to wait a real need to change it (performance issue, or any issue when a model refactor will ease to solve the problem)

By Jean-Baptiste on 2019-08-05T12:46:35 (imported from GitLab project)

I don't get your point.

I think we should take this opportunity to do directly the clean way instead of adding technnical debt.