search

IntelLabs / vdms

VDMS: Your Favorite Visual Data Management System
MIT License
84 stars 31 forks source link

QueryHandler doesn't really check individual command return status #107

Open vishakha041 opened 5 years ago

vishakha041 commented 5 years ago

Line 398 in src/QueryHandler.cc which checks for "status" entry in response JSON seems to essentially be a no-op since it does not descend into any command structure. You have to index into the data structure with the command name if you actually want to check status per command.

ifadams commented 3 months ago

@cwlacewe this might want to be bumped in priority. We generally have issues with lack of clarity/good error handling.