Github user rvesse commented on the issue:

    https://github.com/apache/jena/pull/385
  
    Looks like a really nice extension to the current machinery!
    
    I notice there is a unit test for the assembler support but no unit tests 
for using some of these new features, would it be possible to include some of 
those to verify that this actually allows addressing the use cases that led to 
this PR?
    
    One other thought is whether the explicit param type definitions are 
necessary for the simpler cases (integer, boolean) or whether we could just 
rely on literal datatypes to reduce the verbosity of the configuration for 
those simpler cases?  For the complex cases like sets, token streams, analysers 
etc then explicit types makes total sense.


---

Reply via email to