-
Notifications
You must be signed in to change notification settings - Fork 16
Add remaining operations to picasso #46
Comments
Can you list them here and what you should think of the flow. |
Operations like db ls which use the einstein code directly without sending a REST request to picasso. This will allow us to detach the CLI as you previously pointed out. Also these commands should work only for admins and end users will get a 403 |
Can you be a bit more elaborate. I am kind of lost here. So, are you telling that once we separate will it work or it won't? |
Right now db ls is directly calling the method in BMI object. After adding to picasso then it will become an RPC call. |
@chemistry-sourabh Can you document these TODO operations somewhere? |
All the operations that are calling BMI directly in CLI |
@chemistry-sourabh can we close this? |
There are certain admin level operations that were not added to picasso due to security reasons. These operations should be added as path of the rest API, but must work only for admins.
The text was updated successfully, but these errors were encountered: