Skip to content

Conversation

@srowen
Copy link
Member

@srowen srowen commented Oct 14, 2014

Avoid overflow in computing n*(n+1)/2 as much as possible; throw explicit error when Gramian computation will fail due to negative array size; warn about large result when computing Gramian too

…icit error when Gramian computation will fail due to negative array size; warn about large result when computing Gramian too
@SparkQA
Copy link

SparkQA commented Oct 14, 2014

QA tests have started for PR 2801 at commit b4e6d92.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Oct 14, 2014

QA tests have finished for PR 2801 at commit b4e6d92.

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

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/21723/
Test PASSed.

@mengxr
Copy link
Contributor

mengxr commented Oct 14, 2014

LGTM. Merged into master. Thanks!

@asfgit asfgit closed this in 56096db Oct 14, 2014
@srowen srowen deleted the SPARK-3803 branch October 16, 2014 13:53
MaxGekk added a commit to MaxGekk/spark that referenced this pull request Jul 7, 2018
I added a recommendation for increasing broadcast timeout. This sentence is added to existing error message:

```
You can increase the timeout for broadcasts via ${SQLConf.BROADCAST_TIMEOUT.key}
```

Author: Maxim Gekk <maxim.gekk@databricks.com>

Closes apache#2801 from MaxGekk/broadcast-error-message.
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

Successfully merging this pull request may close these issues.

4 participants