search

play-project / play

The main repository, used for website and issues
2 stars 0 forks source link

Remove useless prefixes from RDF messages #21

Open lpellegr opened 11 years ago

lpellegr commented 11 years ago

The RDF payloads embed by the WS-Notification notification messages should contain only the prefixes that are used.

antonioaversa commented 11 years ago

@lpellegr do you mean from Roland's Java classes, or from whatever RDF event instances source? In the second case: might this issue somehow affect PLAY performances? If yes I will make smarter my templates to accomplish this goal.

lpellegr commented 11 years ago

I would say from both if not everybody is using the Roland's Java classes, otherwise the first solution is sufficient. Not sure that it improves the performances (its only ~10 lines per event) but it will save some bandwidth for sure after some days.

stuehmer commented 11 years ago

Currently there is no code to automatically remove unused prefixes. The Jena people said this: http://tech.groups.yahoo.com/group/jena-dev/message/4876

I'm afraid that finding out which prefixes are unused is your problem. (Some people want to preserve prefixes in the output even when there's no use of them in the model, because they do read-update-write cycles on the file, and want the prefixes available for when they are used.)

We will need to code this, if necessary.

The prefixes which are defined (using my sdk Java classes are defined here https://svn.petalslink.org/svnroot/trunk/research/projects/play/play-commons/play-commons-constants/src/main/java/eu/play_project/play_commons/constants/Namespace.java ).

antonioaversa commented 11 years ago

Yes, basically the same happens for my templates, which it need a few working days to be extended implementing such a feature. @lpellegr If this improvement doesn't greatly increase EventCloud time performances (I think bandwidth is not right now our mission critical issue) I would pospone after the review, and I think it could be the same for Roland (@stuehmer).