Skip to content

Conversation

@marmbrus
Copy link
Contributor

Turns out that inner classes of inner objects are referenced directly, and thus moving it will break binary compatibility.

@andrewor14
Copy link
Contributor

LGTM

@SparkQA
Copy link

SparkQA commented Aug 18, 2015

Test build #41150 has finished for PR 8281 at commit 541d9a0.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • implicit class StringToColumn(val sc: StringContext)

asfgit pushed a commit that referenced this pull request Aug 18, 2015
…tion

Turns out that inner classes of inner objects are referenced directly, and thus moving it will break binary compatibility.

Author: Michael Armbrust <[email protected]>

Closes #8281 from marmbrus/binaryCompat.

(cherry picked from commit 80cb25b)
Signed-off-by: Michael Armbrust <[email protected]>
@marmbrus
Copy link
Contributor Author

Merged to master and 1.5

@asfgit asfgit closed this in 80cb25b Aug 18, 2015
@marmbrus marmbrus deleted the binaryCompat branch March 8, 2016 00:02
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