Skip to content

Conversation

@sarutak
Copy link
Member

@sarutak sarutak commented Oct 11, 2016

What changes were proposed in this pull request?

In programming-guide.md, the url which links to AccumulatorV2 says api/scala/index.html#org.apache.spark.AccumulatorV2 but api/scala/index.html#org.apache.spark.util.AccumulatorV2 is correct.

How was this patch tested?

manual test.

@SparkQA
Copy link

SparkQA commented Oct 11, 2016

Test build #66769 has finished for PR 15439 at commit 6231206.

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

@rxin
Copy link
Contributor

rxin commented Oct 12, 2016

Thanks - merging in master/2.0.

asfgit pushed a commit that referenced this pull request Oct 12, 2016
… is incorrect.

## What changes were proposed in this pull request?

In `programming-guide.md`, the url which links to `AccumulatorV2` says `api/scala/index.html#org.apache.spark.AccumulatorV2` but `api/scala/index.html#org.apache.spark.util.AccumulatorV2` is correct.

## How was this patch tested?
manual test.

Author: Kousuke Saruta <sarutak@oss.nttdata.co.jp>

Closes #15439 from sarutak/SPARK-17880.

(cherry picked from commit b512f04)
Signed-off-by: Reynold Xin <rxin@databricks.com>
@asfgit asfgit closed this in b512f04 Oct 12, 2016
uzadude pushed a commit to uzadude/spark that referenced this pull request Jan 27, 2017
… is incorrect.

## What changes were proposed in this pull request?

In `programming-guide.md`, the url which links to `AccumulatorV2` says `api/scala/index.html#org.apache.spark.AccumulatorV2` but `api/scala/index.html#org.apache.spark.util.AccumulatorV2` is correct.

## How was this patch tested?
manual test.

Author: Kousuke Saruta <sarutak@oss.nttdata.co.jp>

Closes apache#15439 from sarutak/SPARK-17880.
@sarutak sarutak deleted the SPARK-17880 branch June 4, 2021 20:45
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.

3 participants