-
Notifications
You must be signed in to change notification settings - Fork 996
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
Ingestion with Spark: Job Management for Beam Spark Runner #362
Comments
This can be labeled "enhancement" I think. I'd be glad to carry the triage role responsibly |
Agreed. I want to move to a world where Feast Core requires no privileged access and is just a simple registry. We can contain the complexity in either serving or in jobs (setting aside dependency management for a second).
Does this mean you are going for a different client, or a different server (non Livy)?
This does seem tricky. I dont have enough familiarity with the variation in deployment methods above to know how we can deal with this. My instinct is to either reign in the complexity by standardizing on a single solution (Flink or a Spark implementation), or by providing an interface or extension layer to development teams. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Seems like https://github.com/spark-jobserver/spark-jobserver address both concerns in this issue and is used by datastax |
This might be another option for modularity using a spark operator - https://github.com/GoogleCloudPlatform/spark-on-k8s-operator/blob/master/sparkctl/README.md#create |
Closing this issue. We have ingestion support for Spark with ERM, Dataproc, and Spark on K8s. |
We would like to run ingestion on Spark (Streaming), i.e. with the Beam Spark Runner. Thus, an implementation of Feast's job management is needed.
There are a couple of factors that make this a bit less straightforward than Google Cloud Dataflow:
* Other than starting a SparkContext connected to the remote cluster, in-process in Feast Core. I feel that isn't workable for a number of reasons, not least of which are heavy dependencies on Spark as a library, and the lifecycle of streaming ingestion jobs being unnecessarily coupled to that of the Feast Core instance.
Planned Approach
Job Management
We initially plan to implement
JobManager
usingthe Java client library forApache Livy, a REST interface to Spark. This will use only an HTTP client, so it is light on dependencies and shouldn't get in the way of alternativeJobManager
s for Spark, should another organization wish to implement one for something other than Livy. (Edit: turns out that Livy'slivy-http-client
artifact still depends on Spark as a library, it's not a plain REST client, so we'll avoid that…)We have internal experience and precedent using Livy, but not for Spark Streaming applications, so we have some uncertainties about whether it can work well. In case that it doesn't, we'll probably look to try spark-jobserver which does explicitly claim support for Streaming jobs.
Ingestion Job Artifact
We're a bit less certain about how users should get the Feast ingestion Beam job artifact to their Spark cluster, due to the above mentioned variation in deployments.
Roughly speaking, Feast Ingestion would be packaged as an assembly JAR that includes
beam-runners-spark
as well. So, a newingestion-spark
module may be added to the Maven build which is simply a POM for doing just that.Deployment itself may then need to rely on documentation.
Beam Spark Runner
A minor note, but we will use the "legacy", non-portable Beam Spark Runner. As the Beam docs cover, the runner based on Spark Structured Streaming is incomplete and only supports batch jobs, and the non-portable runner is still recommended for Java-only needs.
In theory this is runtime configuration for Feast users: if they want to try the portable runner, it should be possible, but we'll most likely be testing with the non-portable one.
cc @smadarasmi
Reference issues to keep tabs on during implementation: #302, #361.
The text was updated successfully, but these errors were encountered: