[SPARK-10038] [SQL] fix bug in generated unsafe projection when there is binary in ArrayData

[davies]

The type for array of array in Java is slightly different than array of others.

cc @cloud-fan

[JoshRosen]

Jenkins, retest this please.

[SparkQA]

Test build #41054 has finished for PR 8250 at commit 30e9182.

• This patch fails Scala style tests.
• This patch merges cleanly.
• This patch adds no public classes.

[SparkQA]

Test build #1636 has finished for PR 8250 at commit 30e9182.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds the following public classes (experimental):
  • case class FilterNode(condition: Expression, child: LocalNode) extends UnaryLocalNode
  • abstract class LocalNode extends TreeNode[LocalNode]
  • abstract class LeafLocalNode extends LocalNode
  • abstract class UnaryLocalNode extends LocalNode
  • case class ProjectNode(projectList: Seq[NamedExpression], child: LocalNode) extends UnaryLocalNode
  • case class SeqScanNode(output: Seq[Attribute], data: Seq[InternalRow]) extends LeafLocalNode
  • public final class UTF8String implements Comparable<UTF8String>, Externalizable

[SparkQA]

Test build #41058 has finished for PR 8250 at commit 31141d1.

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

[cloud-fan]

Ah good catch! The change LGTM.

Not related to this PR, but as we have more and more hack(equals, hashCode, etc.) for binary type, should we create a wrapper class for it?

[rxin]

I've merged this. Thanks!