Skip to content
This repository has been archived by the owner on May 23, 2024. It is now read-only.

Several integration tests from QueryServiceTest are timing out #613

Open
gaurav opened this issue Jan 24, 2023 · 3 comments
Open

Several integration tests from QueryServiceTest are timing out #613

gaurav opened this issue Jan 24, 2023 · 3 comments

Comments

@gaurav
Copy link
Member

gaurav commented Jan 24, 2023

I discovered this running the integration tests locally. They also timeout on cam-kp-api-dev.renci.org but run successfully on cam-kp-api. This includes:

  • {"message":{"query_graph":{"nodes":{"n0":{"ids":["NCBIGene:3105","NCBIGene:3106","NCBIGene:64167","NCBIGene:60412","NCBIGene:26191"],"categories":["biolink:GeneOrGeneProduct"]},"n1":{"categories":["biolink:GeneOrGeneProduct"]}},"edges":{"e0":{"predicates":["biolink:affects"],"subject":"n0","object":"n1"}}}}}
  • {"message":{"query_graph":{"nodes":{"n0":{"categories":["biolink:GeneOrGeneProduct"]},"n1":{"categories":["biolink:BiologicalProcess"]}},"edges":{"e0":{"predicates":["biolink:has_participant"],"subject":"n1","object":"n0"}}}}}

I've tried restarting cam-kp-sparql but that didn't solve the problem.

@gaurav
Copy link
Member Author

gaurav commented Jan 27, 2023

I've tried increasing the memory on cam-kp-sparql from 32Gi to 64Gi, but that didn't fix it.

It might be related to this error I've been seeing in the CAM-KP SPARQL log -- maybe I've misconfigured something? The Helm chart I'm using is at helxplatform/translator-devops#526

gaurav added a commit to ExposuresProvider/cam-pipeline that referenced this issue Jan 27, 2023
This PR adds the Kubernetes CronJob file to the repository. It moves one of the steps from the Kubernetes file into the Makefile. It seems to be mostly working -- I've tested one of the files generated by this job and it produces a small number of errors on CAM-KP-API (#80, ExposuresProvider/cam-kp-api#613) -- and I'd rather not delay this any further so that I can move on to trying to upgrade cam-pipeline to Biolink 3.
@gaurav
Copy link
Member Author

gaurav commented Jan 30, 2023

Okay, I've found an example SPARQL query that works fine on the old triplestore, but that times out on the new triplestore. @balhoff Do you have any idea why the two Blazegraph databases might react so differently to this?

@balhoff
Copy link
Contributor

balhoff commented Jan 31, 2023

@gaurav not exactly related to the performance issue, but I don't think that ?n0_class and ?n1_class should be projected out of the sub-select (and also shouldn't be in the GROUP BY).

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

No branches or pull requests

2 participants