Skip to content
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

support protobuf #957

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

support protobuf #957

wants to merge 8 commits into from

Conversation

lxning
Copy link
Contributor

@lxning lxning commented Oct 18, 2020

Before or while filing an issue please feel free to join our slack channel to get in touch with development team, ask questions, find out what's cooking and more!

Issue #, if available:

951

Description of changes:

  1. define inference message proto
  • add inference.proto
  • update gradle to support protobuf build
  1. support protobuf encode/decode in inference channel
  • java code changes in worker data flow (ie. wlm dir)
  • java code changes in http handler (ie. http dir)
  • java code changes in tools (ie. util dir)
  1. add unit test cases
  • testPingProto
  • testPredictionsProto
  • testPredictionsModelNotFoundProto
  1. fix mms-ci-build issues
  • ci/Dockerfile.python3.6
  • mms/tests/unit_tests/test_beckend_metric.py
  • mms/tests/unit_tests/test_worker_service.py

Testing done:

Run unit test at local.

To run CI tests on your changes refer README.md

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@lxning lxning changed the title Lxning protobuf support protobuf Oct 18, 2020
@lxning lxning requested a review from maaquib October 18, 2020 17:45
@lxning lxning linked an issue Oct 18, 2020 that may be closed by this pull request
@lxning lxning self-assigned this Oct 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

define inference proto
1 participant