Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21535
If we only test interpreted path on selected test suites, e.g.,
`org.apache.spark.sql.catalyst.encoders.*`, I think we won't increase too much
test time. I feel that we should not make it different
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/21535
I am not sure whether the increased time is acceptable or not.
If it is not accepted, how about adding an option to control do test in
interpreter mode. Then, we test only compiled code path
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92509/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21535
**[Test build #92509 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92509/testReport)**
for PR 21535 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21535
**[Test build #92509 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92509/testReport)**
for PR 21535 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/608/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21535
retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92508/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21535
**[Test build #92508 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92508/testReport)**
for PR 21535 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/607/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21535
**[Test build #92508 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92508/testReport)**
for PR 21535 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21535
retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21535
ping @hvanhovell @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21535
ping @hvanhovell and cc @cloud-fan too.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21535
@hvanhovell Do you think we should test interpreted paths against
DatasetSuite and DataFrameSuite?
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91807/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21535
**[Test build #91807 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91807/testReport)**
for PR 21535 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/115/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/4005/
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21535
@hvanhovell Added tests for interpreted encoders.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21535
**[Test build #91807 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91807/testReport)**
for PR 21535 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21535
@hvanhovell I observed the testing time, roughly for a same test,
additional interpreted path will add 50~120% time.
Interpreted encoders? Do you mean to only test interpreted path for
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91707/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21535
**[Test build #91707 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91707/testReport)**
for PR 21535 at commit
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/21535
@viirya how much time this add to the overall testing time?
If this is too much then we should perhaps just test interpreted encoders.
WDYT?
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21535
**[Test build #91707 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91707/testReport)**
for PR 21535 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/44/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3934/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21535
retest this please...
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91699/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21535
**[Test build #91699 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91699/testReport)**
for PR 21535 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21535
**[Test build #91699 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91699/testReport)**
for PR 21535 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/40/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3930/
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21535
retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91692/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21535
**[Test build #91692 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91692/testReport)**
for PR 21535 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3927/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21535
**[Test build #91692 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91692/testReport)**
for PR 21535 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/37/
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21535
retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91687/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21535
**[Test build #91687 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91687/testReport)**
for PR 21535 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3925/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/35/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21535
**[Test build #91687 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91687/testReport)**
for PR 21535 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21535
cc @cloud-fan @hvanhovell @kiszk @mgaido91 @maropu
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
66 matches
Mail list logo
