It seems PinotAdministrator doesn't return exit co...
# generala
Adrian Cole
09/04/2020, 5:41 AMIt seems PinotAdministrator doesn't return exit code 1 on failure.. this makes use in shell scripts difficult
Adrian Cole
09/04/2020, 5:43 AMI will double check but for example I found a script that had exit status enforced, but continued anyway
Adrian Cole
09/04/2020, 5:43 AMAdrian Cole
09/04/2020, 5:45 AM Copy code
pinot | 2020/09/04 05:45:11.326 INFO [AddTableCommand] [main] {"code":400,"error":"Invalid table config for table rawServiceView_REALTIME: Failed to find instances with tag: DefaultTenant_BROKER for table: rawServiceView_REALTIME"}Adrian Cole
09/04/2020, 5:45 AMthis didn't quit out and I would have expected it to..
x
Xiang Fu
09/04/2020, 4:49 PM
Can you provide the cmd? We will fix it
Xiang Fu
09/04/2020, 4:49 PM
Or just file an issue on GitHub and tag as big
Xiang Fu
09/04/2020, 4:50 PM
Bug*
a
Adrian Cole
09/07/2020, 1:08 AMWhat I experienced was a glitch in running the same command, but later we switched to just use REST api as it is less resource usage for the same outcome and also HTTP responds naturally https://github.com/hypertrace/pinot/blob/main/pinot-servicemanager/docker-bin/install-schema#L38-L43