search

overture-stack / maestro

Indexing software to connect SONG and Arranger through Elasticsearch
https://www.overture.bio/documentation/maestro/
GNU Affero General Public License v3.0
0 stars 0 forks source link

Maestro fails to re-index the updates/changes in metadata for existing analysis #252

Closed ghost closed 1 year ago

ghost commented 2 years ago

Expected Behaviour

Maestro should re-index the changes in Elastic Search for existing doc.

Actual Behaviour

Maestro doesn't re-index the changes in Elastic Search for existing doc.

Possible Fix

Change the Bulk Update Query and or other related query for ES to update for existing doc

Steps to Reproduce

  1. Update Analysis - https://song.dev.cancogen.cancercollaboratory.org/swagger-ui.html#/Analysis/updateAnalysisUsingPUT
  2. Index Analysis - https://maestro.dev.cancogen.cancercollaboratory.org/webjars/swagger-ui/index.html?configUrl=/v3/api-docs/swagger-config#/Indexing/indexAnalysis

Updates/changes will not be reflected in ES

lrivera-oicr commented 2 years ago

changed the Bulk update script to overwrite the source without altering the purpose of the previous script.