org.apache.spark.ml.parity.SparkParityBase.spark should be a method

Open
#856 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
55/100
Issue type
Bug
Clarity
Clearly specified
Activity status
Stale
Tech stack
scala, spark

Research direction

Search the repository for the SparkParityBase class and inspect its spark member. Change the member declaration as described so child classes can override it without eagerly creating a Spark session, then run the relevant parity or test suite to verify the override behavior.

Written by the indexing model from the issue text.

Description

waiting for input

current abstract class SparkParityBase has a variable
val spark = SparkEnv.spark
Even user override it in child class, it will still create a spark session.
change it to
def spark = SparkEnv.spark will allow user to actually override it

Dominant language
Scala
Stars
1.5k
Forks
313
Avg merge
22m
Merged PRs (30d)
1

Getting set up

Open in Codespaces

Starts the project's dev container in your browser, under your own GitHub account.

  • No Dockerfile or Docker Compose file
  • No pull request template
  • No contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from combust/mleap

All issues in combust/mleap

Similar issues

More Scala issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.