-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Included crawler service demo #92
Conversation
@chandrams Can you please all of the common code between the various local monitoring demos to one place and not duplicate the code. I dont think we need most of the code that is present in this PR |
Signed-off-by: Chandrakala Subramanyam <[email protected]>
…ice changes Signed-off-by: Chandrakala Subramanyam <[email protected]>
…utput Signed-off-by: Chandrakala Subramanyam <[email protected]>
Signed-off-by: Chandrakala Subramanyam <[email protected]>
Signed-off-by: Chandrakala Subramanyam <[email protected]>
Signed-off-by: Saad Khan <[email protected]>
Signed-off-by: Chandrakala Subramanyam <[email protected]>
Signed-off-by: Chandrakala Subramanyam <[email protected]>
Signed-off-by: Chandrakala Subramanyam <[email protected]>
ab58c8a
to
61a82d8
Compare
Signed-off-by: Chandrakala Subramanyam <[email protected]>
@dinogun - I have updated the bulk API as per the latest code changes and moved the common code out. Please review. |
@chandrams Please update the README.md and add content on the bulk API demo goals. Ideally bulk APIs are going to be more useful to users than the individual APIs as users can get recommendations for individual workloads or groups of them by fiddling with the Also in a separate PR, we can have additional examples of various bulk input json's that will enable users to get the right kind of recommendations |
Signed-off-by: Chandrakala Subramanyam <[email protected]>
@dinogun - I have updated the readme for bulk demo, please review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.