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

[MINOR][DOC] Documentation on JVM options for SBT #23615

Closed
wants to merge 3 commits into from

Conversation

da-liii
Copy link
Contributor

@da-liii da-liii commented Jan 22, 2019

What changes were proposed in this pull request?

Documentation and .gitignore

How was this patch tested?

Manual test that SBT honors the settings in .jvmopts if present

docs/building-spark.md Outdated Show resolved Hide resolved
@da-liii
Copy link
Contributor Author

da-liii commented Jan 22, 2019

@srowen I've updated the doc. Please tell me if there are any errors. (A bit embarrassing 囧 )

@SparkQA
Copy link

SparkQA commented Jan 22, 2019

Test build #101539 has finished for PR 23615 at commit fe402ee.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Jan 22, 2019

Test build #101543 has finished for PR 23615 at commit c6e6ef3.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@dongjoon-hyun
Copy link
Member

dongjoon-hyun commented Jan 22, 2019

Hi, @sadhen . Could you update the section, How was this patch tested?, in the PR description?

## How was this patch tested?
None.

@srowen
Copy link
Member

srowen commented Jan 23, 2019

I think it's fine; I tested that indeed it causes SBT to use the supplied settings. I'll just edit the description.

@dongjoon-hyun
Copy link
Member

Thanks, @srowen .

@srowen
Copy link
Member

srowen commented Jan 23, 2019

Merged to master

@srowen srowen closed this in 9d2a115 Jan 23, 2019
jackylee-ch pushed a commit to jackylee-ch/spark that referenced this pull request Feb 18, 2019
## What changes were proposed in this pull request?

Documentation and .gitignore

## How was this patch tested?

Manual test that SBT honors the settings in .jvmopts if present

Closes apache#23615 from sadhen/impr/gitignore.

Authored-by: Darcy Shen <sadhen@zoho.com>
Signed-off-by: Sean Owen <sean.owen@databricks.com>
@nchammas
Copy link
Contributor

Is there any reason not to just check in a .jvmopts file that captures these settings, and remove the corresponding entry from .gitignore? Seems like that would be better, since it saves contributors from a manual step. If appropriate, I can submit a PR for this.

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