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

Enhancements for 3rd framework integration #8

Closed
leftnoteasy opened this issue Apr 11, 2018 · 2 comments
Closed

Enhancements for 3rd framework integration #8

leftnoteasy opened this issue Apr 11, 2018 · 2 comments
Assignees

Comments

@leftnoteasy
Copy link

@tobegit3hub, first of all, thanks for working on this framework, really appreciated!

While working on a prototype of YARN submarine project (YARN-8135). I saw this project and wanna to integrate this project into the model serving path.

There're a couple of requirements so far I can see:

  1. Be able to read HDFS files.
    This is P0 requirement since model could be very large.

  2. Allow recursively detect saved model path.
    Existing model serving expects folder with version numbers exist under the specified model path, it might be convenient for end users if the framework can recursively looking for models for serving.
    This is a P1 requirement.

  3. Expose REST APIs to get models-signatures / configs / status, etc.
    Existing simple_tensorflow_serving only exposes POST APIs for model serving, it's better to expose GET APIs as well since simple_tensorflow_serving might be launched in a different machine. We can expose information like:
    a. Status of the service. (including model name, paths, statuses, etc.)
    b. Configs of the service.
    c. Model signatures.
    d. Generate clients examples (with proper endpoint).

Thanks again!

@tobegit3hub
Copy link
Owner

tobegit3hub commented Apr 12, 2018

Thanks @leftnoteasy for the great suggestions.

We have considered supporting HDFS files and more RESTful APIs. We're really welcome to any feedback and looking forward to contributing this for Hadoop ecosystem 😃

@tobegit3hub tobegit3hub self-assigned this Jul 4, 2018
@tobegit3hub
Copy link
Owner

Thanks to the help of community, most of these requested features are implemented.

  1. Support reading remote models in HDFS including Kerberos supported.
  2. Arbitrary recursively paths will not be supported because it may be ambiguous if we have multiple models in the same sub-directory. We want user to specify the complete model path just like TensorFlow Serving.
  3. Now we have more APIs for use to get model signature, generate example json and client in multiple programming languages.

I hope this helps and find an way to be part of the hadoop ecosystem. This issue may be closed and feel free to re-open if we have any other question.

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

No branches or pull requests

2 participants