zjffdu commented on issue #3480: [ZEPPELIN-3644] SPARQL-Interpreter support via
Apache Jena ARQ
URL: https://github.com/apache/zeppelin/pull/3480#issuecomment-552171907
LGTM, will merge if no more comment
This is an
zjffdu commented on issue #3480: [ZEPPELIN-3644] SPARQL-Interpreter support via
Apache Jena ARQ
URL: https://github.com/apache/zeppelin/pull/3480#issuecomment-549822879
I think it is fine to add this setting until there's another implementation.
zjffdu commented on issue #3480: [ZEPPELIN-3644] SPARQL-Interpreter support via
Apache Jena ARQ
URL: https://github.com/apache/zeppelin/pull/3480#issuecomment-548634503
> > Thanks for the contribution @hsteinmueller , this PR LGTM, could you
paste the travis build link ?
>
>
zjffdu commented on issue #3480: [ZEPPELIN-3644] SPARQL-Interpreter support via
Apache Jena ARQ
URL: https://github.com/apache/zeppelin/pull/3480#issuecomment-548633999
@hsteinmueller Could you update the pom.xml to resolve the conflict ?
zjffdu commented on issue #3480: [ZEPPELIN-3644] SPARQL-Interpreter support via
Apache Jena ARQ
URL: https://github.com/apache/zeppelin/pull/3480#issuecomment-546163834
IIUC, `jena` is one implementation of `sparql`. Since interpreter text
`%sparql` is user facing, I would prefer to
zjffdu commented on issue #3480: [ZEPPELIN-3644] SPARQL-Interpreter support via
Apache Jena ARQ
URL: https://github.com/apache/zeppelin/pull/3480#issuecomment-545959511
Thanks for the contribution @hsteinmueller , this PR LGTM, could you paste
the travis build link ?