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

Correct reference to Offset class #21387

Closed
wants to merge 1 commit into from
Closed

Conversation

mojodna
Copy link
Contributor

@mojodna mojodna commented May 21, 2018

This is a documentation-only correction; org.apache.spark.sql.sources.v2.reader.Offset is actually org.apache.spark.sql.sources.v2.reader.streaming.Offset.

@HyukjinKwon
Copy link
Member

It's okay. mind taking a quick look for the same instances while we are here?

@mojodna
Copy link
Contributor Author

mojodna commented May 22, 2018

I don't see any more mentions of the incorrect package.

@HyukjinKwon
Copy link
Member

ok to test

@SparkQA
Copy link

SparkQA commented May 22, 2018

Test build #90987 has finished for PR 21387 at commit 81a5d3d.

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

@HyukjinKwon
Copy link
Member

Merged to master and branch-2.3.

asfgit pushed a commit that referenced this pull request May 23, 2018
This is a documentation-only correction; `org.apache.spark.sql.sources.v2.reader.Offset` is actually `org.apache.spark.sql.sources.v2.reader.streaming.Offset`.

Author: Seth Fitzsimmons <[email protected]>

Closes #21387 from mojodna/patch-1.

(cherry picked from commit 00c13cf)
Signed-off-by: hyukjinkwon <[email protected]>
@asfgit asfgit closed this in 00c13cf May 23, 2018
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