[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19882 **[Test build #84443 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84443/testReport)** for PR 19882 at commit [`5f2025a`](https://github.com/apache/spark/commit/5f2025ada40d146411b5e14ddf37f3fccc4cae97). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/19882 Hi, @cloud-fan , @gatorsmile , @HyukjinKwon , @viirya . This is a test case restructure after https://github.com/apache/spark/pull/19651 . --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19882 **[Test build #84443 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84443/testReport)** for PR 19882 at commit [`5f2025a`](https://github.com/apache/spark/commit/5f2025ada40d146411b5e14ddf37f3fccc4cae97). * This patch passes all tests. * This patch merges cleanly. * This patch adds the following public classes _(experimental)_: * `class OrcFilterSuite extends OrcTest with SharedSQLContext ` * `implicit class IntToBinary(int: Int) ` * `case class OrcParData(intField: Int, stringField: String)` * `case class OrcParDataWithKey(intField: Int, pi: Int, stringField: String, ps: String)` * `abstract class OrcPartitionDiscoveryTest extends OrcTest ` * `class OrcPartitionDiscoverySuite extends OrcPartitionDiscoveryTest with SharedSQLContext ` * `case class AllDataTypesWithNonPrimitiveType(` * `case class BinaryData(binaryData: Array[Byte])` * `case class Contact(name: String, phone: String)` * `case class Person(name: String, age: Int, contacts: Seq[Contact])` * `abstract class OrcQueryTest extends OrcTest ` * ` test(\"Creating case class RDD table\") ` * ` test(\"save and load case class RDD with `None`s as orc\") ` * `class OrcQuerySuite extends OrcQueryTest with SharedSQLContext ` * `case class OrcData(intField: Int, stringField: String)` * `abstract class OrcSuite extends OrcTest with BeforeAndAfterAll ` * `class OrcSourceSuite extends OrcSuite with SharedSQLContext ` * `abstract class OrcTest extends QueryTest with SQLTestUtils ` * `class HiveOrcFilterSuite extends OrcTest with TestHiveSingleton ` * `implicit class IntToBinary(int: Int) ` * `class HiveOrcPartitionDiscoverySuite extends OrcPartitionDiscoveryTest with TestHiveSingleton ` * `class HiveOrcQuerySuite extends OrcQueryTest with TestHiveSingleton ` * `class HiveOrcSourceSuite extends OrcSuite with TestHiveSingleton ` * `class HiveOrcHadoopFsRelationSuite extends OrcHadoopFsRelationSuite ` --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19882 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19882 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84443/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/19882 Whoa big class list. Will take a look soon within tomorrow as well. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/19882 Thank you so much, @HyukjinKwon ! --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/19882 > OrcTest will provide common helper functions and def format: String. Instead of having `def format: String`, can we just add `beforeAll` and `afterAll` in the test suites to set the `ORC_IMPLEMENTATION`? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/19882 It has more lines, doesn't it? In any way, we need helper functions. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/19882 ok maybe have a `def orcImp: String`, which can be `native` or `hive`. Then we can put the `beforeAll` and `afterAll` in `OrcTest`. It can avoid changing the test code from `spark.read.orc` to `spark.read.format(format)` --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/19882 Okay. No problem. Thanks, @cloud-fan . --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19882 **[Test build #84524 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84524/testReport)** for PR 19882 at commit [`baec5fe`](https://github.com/apache/spark/commit/baec5fece95896a28127cd2ac014582874ed5e28). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19882 **[Test build #84524 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84524/testReport)** for PR 19882 at commit [`baec5fe`](https://github.com/apache/spark/commit/baec5fece95896a28127cd2ac014582874ed5e28). * This patch passes all tests. * This patch merges cleanly. * This patch adds the following public classes _(experimental)_: * `class OrcFilterSuite extends OrcTest with SharedSQLContext ` * `implicit class IntToBinary(int: Int) ` * `case class OrcParData(intField: Int, stringField: String)` * `case class OrcParDataWithKey(intField: Int, pi: Int, stringField: String, ps: String)` * `abstract class OrcPartitionDiscoveryTest extends OrcTest ` * `case class AllDataTypesWithNonPrimitiveType(` * `case class BinaryData(binaryData: Array[Byte])` * `case class Contact(name: String, phone: String)` * `case class Person(name: String, age: Int, contacts: Seq[Contact])` * `abstract class OrcQueryTest extends OrcTest ` * ` test(\"Creating case class RDD table\") ` * ` test(\"save and load case class RDD with `None`s as orc\") ` * `class OrcQuerySuite extends OrcQueryTest with SharedSQLContext ` * `case class OrcData(intField: Int, stringField: String)` * `abstract class OrcSuite extends OrcTest with BeforeAndAfterAll ` * `class OrcSourceSuite extends OrcSuite with SharedSQLContext ` * `abstract class OrcTest extends QueryTest with SQLTestUtils with BeforeAndAfterAll ` * `class HiveOrcFilterSuite extends OrcTest with TestHiveSingleton ` * `implicit class IntToBinary(int: Int) ` * `class HiveOrcPartitionDiscoverySuite extends OrcPartitionDiscoveryTest with TestHiveSingleton ` * `class HiveOrcQuerySuite extends OrcQueryTest with TestHiveSingleton ` * `class HiveOrcSourceSuite extends OrcSuite with TestHiveSingleton ` * `class HiveOrcHadoopFsRelationSuite extends OrcHadoopFsRelationSuite ` --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19882 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19882 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84524/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/19882 It's rebased to the master to resolve conflicts. Also, I addressed the comments. Thanks! --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19882 **[Test build #84581 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84581/testReport)** for PR 19882 at commit [`fcb2ccb`](https://github.com/apache/spark/commit/fcb2ccb64813f0fc37e3321c006937fb9cb870e4). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/19882 LGTM BTW. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19882 **[Test build #84581 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84581/testReport)** for PR 19882 at commit [`fcb2ccb`](https://github.com/apache/spark/commit/fcb2ccb64813f0fc37e3321c006937fb9cb870e4). * This patch passes all tests. * This patch merges cleanly. * This patch adds no public classes. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19882 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19882 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84581/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/19882 Thank you so much, @HyukjinKwon ! --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19882 **[Test build #84589 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84589/testReport)** for PR 19882 at commit [`1828571`](https://github.com/apache/spark/commit/18285717edcbefe9a79d51a6fdcb112c3d0bc5c4). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/19882 @HyukjinKwon . The PR code and description is updated. - Update comments - Rename back to the originals The main reason I used prefix `Hive` is for naming consistency, but now it's restored. As you see in the PR description, we need to use `HiveOrcHadoopFsRelationSuite` inconsistently because `OrcHadoopFsRelationSuite` already exists in the same package. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/19882 Yup but I actually received an advise before. I suggested: ``` ``` ``` org.apache.spark.sql.execution.datasources.csv.CSVInferSchema org.apache.spark.sql.execution.datasources.json.JsonInferSchema ``` --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/19882 I actually suggested similarly before: ``` org.apache.spark.sql.execution.datasources.csv.InferSchema org.apache.spark.sql.execution.datasources.json.InferSchema ``` but I actually received an advise at that time and it became as below: ``` org.apache.spark.sql.execution.datasources.csv.CSVInferSchema org.apache.spark.sql.execution.datasources.json.JsonInferSchema ``` I actually liked `Hive` prefix it was easier to distinguish. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/19882 I meant in https://github.com/apache/spark/pull/19882#pullrequestreview-81708676, I liked this `Hive` prefix here so wondered if we could do the same thing for the main codes too. Since this PR only refactors the tests, it's loosely related though. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/19882 Oh. I'll bring back. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/19882 I am sorry, I had to clarify this ahead .. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/19882 Definitely, my bad. For the main code, we can do later in a separate PR if needed. I hope this PR contains tests only. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19882 **[Test build #84591 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84591/testReport)** for PR 19882 at commit [`f56423d`](https://github.com/apache/spark/commit/f56423dd9168e781fad6e02e2aa319d8593d98cc). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19882 **[Test build #84589 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84589/testReport)** for PR 19882 at commit [`1828571`](https://github.com/apache/spark/commit/18285717edcbefe9a79d51a6fdcb112c3d0bc5c4). * This patch passes all tests. * This patch merges cleanly. * This patch adds the following public classes _(experimental)_: * `class OrcFilterSuite extends OrcTest with TestHiveSingleton ` * `implicit class IntToBinary(int: Int) ` * `class OrcPartitionDiscoverySuite extends OrcPartitionDiscoveryTest with TestHiveSingleton ` * `class OrcQuerySuite extends OrcQueryTest with TestHiveSingleton ` * `class OrcSourceSuite extends OrcSuite with TestHiveSingleton ` --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19882 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19882 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84589/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19882 **[Test build #84591 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84591/testReport)** for PR 19882 at commit [`f56423d`](https://github.com/apache/spark/commit/f56423dd9168e781fad6e02e2aa319d8593d98cc). * This patch **fails due to an unknown error code, -9**. * This patch merges cleanly. * This patch adds no public classes. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19882 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84591/ Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19882 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/19882 retest this please --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19882 **[Test build #84599 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84599/testReport)** for PR 19882 at commit [`f56423d`](https://github.com/apache/spark/commit/f56423dd9168e781fad6e02e2aa319d8593d98cc). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19882 **[Test build #84599 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84599/testReport)** for PR 19882 at commit [`f56423d`](https://github.com/apache/spark/commit/f56423dd9168e781fad6e02e2aa319d8593d98cc). * This patch passes all tests. * This patch merges cleanly. * This patch adds no public classes. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19882 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19882 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84599/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/19882 thanks, merging to master! --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19882: [SPARK-22672][SQL][TEST] Refactor ORC Tests
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/19882 Thank you so much, @cloud-fan , @HyukjinKwon , and @gatorsmile ! --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org