Skip to content

Conversation

@cloud-fan
Copy link
Contributor

No description provided.

@cloud-fan
Copy link
Contributor Author

cc @rxin

@cloud-fan cloud-fan changed the title [SQL][minor] remove not needed createCode and createStructCode [SQL][minor] remove the no-longer-necessary createCode and createStructCode Aug 3, 2015
@rxin
Copy link
Contributor

rxin commented Aug 3, 2015

LGTM

@SparkQA
Copy link

SparkQA commented Aug 3, 2015

Test build #39526 has finished for PR 7890 at commit af68d20.

  • This patch fails to build.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • class NaiveBayes(JavaEstimator, HasFeaturesCol, HasLabelCol, HasPredictionCol, HasProbabilityCol,
    • public class UnsafeArrayData extends ArrayData
    • public class UnsafeMapData extends MapData
    • public class UnsafeReaders
    • public static class ArrayWriter
    • public static class MapWriter
    • public class UnsafeWriters
    • public static class DecimalWriter
    • public static class UTF8StringWriter
    • public static class BinaryWriter
    • public static class StructWriter
    • public static class IntervalWriter
    • public static class ArrayWriter
    • public static class MapWriter
    • case class FromUnsafe(child: Expression) extends UnaryExpression
    • class RowOrdering(ordering: Seq[SortOrder]) extends Ordering[InternalRow]
    • case class AtLeastNNulls(n: Int, children: Seq[Expression]) extends Predicate
    • case class AtLeastNNonNullNans(n: Int, children: Seq[Expression]) extends Predicate
    • class DefaultOptimizer extends Optimizer
    • case class PartitioningCollection(partitionings: Seq[Partitioning])
    • case class FilterNullsInJoinKey(

@cloud-fan cloud-fan changed the title [SQL][minor] remove the no-longer-necessary createCode and createStructCode [SPARK-9553][SQL] remove the createCode and createStructCode, and replace the usage of them by createStructCode Aug 3, 2015
@cloud-fan
Copy link
Contributor Author

it's not minor anymore as we need ceateCode some place else...
I have created a JIRA for it.

@SparkQA
Copy link

SparkQA commented Aug 3, 2015

Test build #39537 has finished for PR 7890 at commit d561e42.

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

@cloud-fan cloud-fan changed the title [SPARK-9553][SQL] remove the createCode and createStructCode, and replace the usage of them by createStructCode [SPARK-9553][SQL] remove the createCode and createStructCode, and replace the usage of them by createStructCode2 Aug 3, 2015
@cloud-fan cloud-fan changed the title [SPARK-9553][SQL] remove the createCode and createStructCode, and replace the usage of them by createStructCode2 [SPARK-9553][SQL] remove the no-longer-necessary createCode and createStructCode, and replace the usage Aug 3, 2015
@rxin
Copy link
Contributor

rxin commented Aug 3, 2015

LGTM

@SparkQA
Copy link

SparkQA commented Aug 3, 2015

Test build #39545 has finished for PR 7890 at commit b0cbe2e.

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

@SparkQA
Copy link

SparkQA commented Aug 4, 2015

Test build #39717 has finished for PR 7890 at commit c3b1be3.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Aug 4, 2015

Test build #1330 has finished for PR 7890 at commit c3b1be3.

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

@rxin
Copy link
Contributor

rxin commented Aug 4, 2015

I've merged this. Thanks.

@rxin
Copy link
Contributor

rxin commented Aug 4, 2015

This hasn't been merged yet because there are some problems with asf git.

https://issues.apache.org/jira/browse/INFRA-10079

@rxin
Copy link
Contributor

rxin commented Aug 4, 2015

Merged now.

asfgit pushed a commit that referenced this pull request Aug 4, 2015
…eStructCode, and replace the usage

Author: Wenchen Fan <cloud0fan@outlook.com>

Closes #7890 from cloud-fan/minor and squashes the following commits:

c3b1be3 [Wenchen Fan] fix style
b0cbe2e [Wenchen Fan] remove the createCode and createStructCode, and replace the usage of them by createStructCode

(cherry picked from commit f4b1ac0)
Signed-off-by: Reynold Xin <rxin@databricks.com>
@asfgit asfgit closed this in f4b1ac0 Aug 4, 2015
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