Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AbstractCassandraRdfHector does not stop worker threads after closing (after bulkLoad) #83

Open
GoogleCodeExporter opened this issue Sep 10, 2015 · 4 comments

Comments

@GoogleCodeExporter
Copy link

What steps will reproduce the problem?
1. Open a new CassandraRdfHectorTriple connection
2. Bulk load a TTL file
3. Close the connection

What is the expected output? What do you see instead?
The program should exit normally. Instead the worker threads are running, thus 
the program keeps running.

What version of the product are you using? On what operating system?
CumulusRDF 1.0.1, Cassandra 1.2.16

Please provide any additional information below.

AbstractCassandraRdfHector.close() should shutdown these threads, but it does 
not.

The structure of my program is the following:
crdf = new CassandraRdfHectorTriple(servers, "test");
crdf.open();

crdf.bulkLoad(documentFile, RDFFormat.TURTLE);
-- this creates the following threads:
 - Thread [pool-1-thread-1] (Running)   
 - Thread [pool-1-thread-2] (Running)   
 - Thread [pool-1-thread-3] (Running)   
 - Thread [pool-1-thread-4] (Running)   
 - Thread [pool-1-thread-5] (Running)   

crdf.close();
-- threads still running, however the load was successful



Using the Sesame load method is a workaround, but it is much slower.

Original issue reported on code.google.com by steind...@gmail.com on 27 Jun 2014 at 2:32

@GoogleCodeExporter
Copy link
Author

Hi, nice shot! In 1.0.1 we never shutdown the worker thread pool :) I'ill 
commit a patch as soon as possible

Original comment by a.gazzarini@gmail.com on 28 Jun 2014 at 9:34

@GoogleCodeExporter
Copy link
Author

Original comment by a.gazzarini@gmail.com on 28 Jun 2014 at 9:38

  • Changed state: Started

@GoogleCodeExporter
Copy link
Author

Ok, problem has been fixed. I'll ask to our admin to update the artifacts in 
the downloads page and I'll let you know.

Best, 
Andrea 

Original comment by a.gazzarini@gmail.com on 28 Jun 2014 at 9:55

@GoogleCodeExporter
Copy link
Author

Hi guys, 

sorry - I was quite busy today, so I haven't updated the downloads yet. 

But - if I understood Andrea correctly - this bug has been fixed in the current 
trunk. So, you can just download the war-package from our Jenkins server [1].

I will update the downloads tomorrow.

HTH
Andreas 

[1] 
http://dev.aifb.kit.edu/jenkins/job/CumulusRDF-trunk/ws/target/cumulusrdf-1.0.1.
war

Original comment by andreas.josef.wagner on 30 Jun 2014 at 12:17

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant