Skip to content

Conversation

@rxin
Copy link
Contributor

@rxin rxin commented Nov 18, 2015

I also wrote a test case -- but unfortunately the test case is not working due to SPARK-11795.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually we don't need this assert, encoderFor already did this.

@rxin
Copy link
Contributor Author

rxin commented Nov 18, 2015

I'm going to merge this and fix the comments in a separate pull request.

asfgit pushed a commit that referenced this pull request Nov 18, 2015
…y for maps.

I also wrote a test case -- but unfortunately the test case is not working due to SPARK-11795.

Author: Reynold Xin <[email protected]>

Closes #9784 from rxin/SPARK-11503.

(cherry picked from commit ed8d153)
Signed-off-by: Reynold Xin <[email protected]>
@asfgit asfgit closed this in ed8d153 Nov 18, 2015
@SparkQA
Copy link

SparkQA commented Nov 18, 2015

Test build #46133 has finished for PR 9784 at commit ae96866.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):\n * class HashingTF(override val uid: String)\n * class Interaction @Since(\"1.6.0\") (override val uid: String) extends Transformer\n * class Normalizer(override val uid: String)\n * class SQLTransformer @Since(\"1.6.0\") (override val uid: String) extends Transformer with Writable\n * class Tokenizer(override val uid: String)\n * case class ArrayConversion(elementConversion: JDBCConversion) extends JDBCConversion\n * abstract class JdbcDialect extends Serializable\n

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