Github user anujgandharv commented on the issue:

    https://github.com/apache/jena/pull/227
  
    @osma @ajs6f I have tried the ES Maven plugin but it is throwing me lots of 
errors. I suspect that it is just a wrapper around the embedded elasticsearch. 
I asked the creator to provide a more comprehensive example of using the 
plugin. I am still waiting for his answer.
    
    In the mean time, in the interest of time and getting this functionality, I 
would "please" suggest that we keep the Tests that we have currently which 
serve for basic testing of the functionality and utilize JenaESTextExample.java 
class implements a more comprehensive testing.
    
    In any case, we require a running instance of ElasticSearch. Till the time 
we do not have a mechanism to start and stop an ElasticSearch instance 
automatically, we can start and stop it manually. I have documented how to do 
that in the JenaESTextExample.java.
    
    Rest of the review comments have been incorporated (unless I missed 
something again).
    
    
    Do you guys agree? and can we merge it in?
    
    Thank,
    Anuj Kumar


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to