Skip to content

Conversation

scv119
Copy link
Contributor

@scv119 scv119 commented Apr 25, 2022

Hey @septicmk. Thanks for submitting the rep and contributing to the Ray community. After voting in Ray committers, we concluded that we won't accept REP #4 (Plasma Alternative) at this time. Concretely:

  1. The ray committers are generally in favor of the direction of having a clean interface for plasma.
  2. The main concern is the feasibility and maintainability of the proposed interface; this requires a fully working prototype to validate.
  3. Also, there are questions about if we have fully explored the design spaces and if there exists a better alternative.

Again, thanks for putting up the efforts to come up with the REP. As we discussed offline, Vineyard and Ray team will collaborate in the following mode:

  1. Ray will not support an alternative plasma store at the moment, and there is no guarantee that the interface won't change.
  2. Instead of proposed REP, Vineyard team will prototype and maintain the integration code as addon/plug-in/patch in one of their repos. As they prototype, they might work with Ray team and submit PRs to clean up Ray's plasma interface that brings net benefit to Ray.
  3. If the community accepts the API changes, and a net benefit to Ray can be demonstrated, and the introduction of Vineyard does not complicate the Ray development process, we may reconsider the plasma alternative REP.

@scv119 scv119 changed the title remove rep4 [Ray enhancement] defer rep4 for future consideration. Apr 25, 2022
@ericl ericl merged commit c3a9127 into main Apr 25, 2022
@scv119 scv119 changed the title [Ray enhancement] defer rep4 for future consideration. [Ray enhancement] defer REP4 (plasma alternative) for future consideration. Apr 25, 2022
@scv119
Copy link
Contributor Author

scv119 commented Apr 25, 2022

Original REP #4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants