You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2020/02/09 13:05:56 UTC

[GitHub] [spark] yaooqinn opened a new pull request #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases

yaooqinn opened a new pull request #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases
URL: https://github.com/apache/spark/pull/27510
 
 
   
   ### What changes were proposed in this pull request?
   
   Add round-trip tests for CSV and JSON functions as  https://github.com/apache/spark/pull/27317#discussion_r376745135 asked.
   
   
   ### Why are the changes needed?
   <!--
   Please clarify why the changes are needed. For instance,
     1. If you propose a new API, clarify the use case for a new API.
     2. If you fix a bug, you can clarify why it is a bug.
   -->
   
   improve test coverage
   
   
   ### Does this PR introduce any user-facing change?
   <!--
   If yes, please clarify the previous behavior and the change this PR proposes - provide the console output, description and/or an example to show the behavior difference if possible.
   If no, write 'No'.
   -->
   
   no
   ### How was this patch tested?
   <!--
   If tests were added, say they were added here. Please make sure to add some test cases that check the changes thoroughly including negative and positive cases if possible.
   If it was tested in a way different from regular unit tests, please clarify how you tested step by step, ideally copy and paste-able, so that other reviewers can test and check, and descendants can verify in the future.
   If tests were not added, please describe why they were not added and/or why it was difficult to add.
   -->
   
   add uts

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins removed a comment on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases
URL: https://github.com/apache/spark/pull/27510#issuecomment-583868209
 
 
   Merged build finished. Test PASSed.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases
URL: https://github.com/apache/spark/pull/27510#issuecomment-583984028
 
 
   Merged build finished. Test PASSed.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins removed a comment on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases
URL: https://github.com/apache/spark/pull/27510#issuecomment-583931852
 
 
   Test PASSed.
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/22869/
   Test PASSed.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA removed a comment on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases

Posted by GitBox <gi...@apache.org>.
SparkQA removed a comment on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases
URL: https://github.com/apache/spark/pull/27510#issuecomment-583843517
 
 
   **[Test build #118097 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118097/testReport)** for PR 27510 at commit [`f9759dc`](https://github.com/apache/spark/commit/f9759dcc313e3cb82421fd13361a1aa45234ff99).

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases
URL: https://github.com/apache/spark/pull/27510#issuecomment-583843657
 
 
   Test PASSed.
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/22862/
   Test PASSed.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins removed a comment on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases
URL: https://github.com/apache/spark/pull/27510#issuecomment-583984028
 
 
   Merged build finished. Test PASSed.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] HyukjinKwon commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases

Posted by GitBox <gi...@apache.org>.
HyukjinKwon commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases
URL: https://github.com/apache/spark/pull/27510#issuecomment-583988628
 
 
   Merged to master and branch-3.0.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases
URL: https://github.com/apache/spark/pull/27510#issuecomment-583931849
 
 
   Merged build finished. Test PASSed.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases

Posted by GitBox <gi...@apache.org>.
SparkQA commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases
URL: https://github.com/apache/spark/pull/27510#issuecomment-583868014
 
 
   **[Test build #118097 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118097/testReport)** for PR 27510 at commit [`f9759dc`](https://github.com/apache/spark/commit/f9759dcc313e3cb82421fd13361a1aa45234ff99).
    * This patch passes all tests.
    * This patch merges cleanly.
    * This patch adds no public classes.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases

Posted by GitBox <gi...@apache.org>.
SparkQA commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases
URL: https://github.com/apache/spark/pull/27510#issuecomment-583843517
 
 
   **[Test build #118097 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118097/testReport)** for PR 27510 at commit [`f9759dc`](https://github.com/apache/spark/commit/f9759dcc313e3cb82421fd13361a1aa45234ff99).

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins removed a comment on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases
URL: https://github.com/apache/spark/pull/27510#issuecomment-583868213
 
 
   Test PASSed.
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118097/
   Test PASSed.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases
URL: https://github.com/apache/spark/pull/27510#issuecomment-583843655
 
 
   Merged build finished. Test PASSed.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins removed a comment on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases
URL: https://github.com/apache/spark/pull/27510#issuecomment-583931849
 
 
   Merged build finished. Test PASSed.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases
URL: https://github.com/apache/spark/pull/27510#issuecomment-583868209
 
 
   Merged build finished. Test PASSed.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases

Posted by GitBox <gi...@apache.org>.
SparkQA commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases
URL: https://github.com/apache/spark/pull/27510#issuecomment-583983462
 
 
   **[Test build #118104 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118104/testReport)** for PR 27510 at commit [`746738f`](https://github.com/apache/spark/commit/746738fa93b90591b09699030c8ca3ec9b57c2ac).
    * This patch passes all tests.
    * This patch merges cleanly.
    * This patch adds no public classes.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases

Posted by GitBox <gi...@apache.org>.
SparkQA commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases
URL: https://github.com/apache/spark/pull/27510#issuecomment-583931571
 
 
   **[Test build #118104 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118104/testReport)** for PR 27510 at commit [`746738f`](https://github.com/apache/spark/commit/746738fa93b90591b09699030c8ca3ec9b57c2ac).

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins removed a comment on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases
URL: https://github.com/apache/spark/pull/27510#issuecomment-583843655
 
 
   Merged build finished. Test PASSed.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA removed a comment on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases

Posted by GitBox <gi...@apache.org>.
SparkQA removed a comment on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases
URL: https://github.com/apache/spark/pull/27510#issuecomment-583931571
 
 
   **[Test build #118104 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118104/testReport)** for PR 27510 at commit [`746738f`](https://github.com/apache/spark/commit/746738fa93b90591b09699030c8ca3ec9b57c2ac).

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins removed a comment on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases
URL: https://github.com/apache/spark/pull/27510#issuecomment-583984034
 
 
   Test PASSed.
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118104/
   Test PASSed.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins removed a comment on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases
URL: https://github.com/apache/spark/pull/27510#issuecomment-583843657
 
 
   Test PASSed.
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/22862/
   Test PASSed.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] cloud-fan commented on a change in pull request #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases

Posted by GitBox <gi...@apache.org>.
cloud-fan commented on a change in pull request #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases
URL: https://github.com/apache/spark/pull/27510#discussion_r376843904
 
 

 ##########
 File path: sql/core/src/test/resources/sql-tests/inputs/interval.sql
 ##########
 @@ -222,7 +222,17 @@ select a * 1.1 from values (interval '-2147483648 months', interval '2147483647
 select a / 0.5 from values (interval '-2147483648 months', interval '2147483647 months') t(a, b);
 
 -- interval support for csv and json functions
-SELECT from_csv('1, 1 day', 'a INT, b interval');
-SELECT to_csv(named_struct('a', interval 32 month, 'b', interval 70 minute));
-SELECT from_json('{"a":"1 days"}', 'a interval');
-SELECT to_json(map('a', interval 25 month 100 day 130 minute));
+SELECT
+  from_csv('1, 1 day', 'a INT, b interval') a,
 
 Review comment:
   I think we only need 4 cases:
   1. from_csv with a constant
   2. to_csv with a constant
   3. from_csv(to_csv(...))
   4. to_csv(from_csv(...))
   
   same to json

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases
URL: https://github.com/apache/spark/pull/27510#issuecomment-583931852
 
 
   Test PASSed.
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/22869/
   Test PASSed.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases
URL: https://github.com/apache/spark/pull/27510#issuecomment-583868213
 
 
   Test PASSed.
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118097/
   Test PASSed.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases
URL: https://github.com/apache/spark/pull/27510#issuecomment-583984034
 
 
   Test PASSed.
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118104/
   Test PASSed.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] yaooqinn commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases

Posted by GitBox <gi...@apache.org>.
yaooqinn commented on issue #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases
URL: https://github.com/apache/spark/pull/27510#issuecomment-583843336
 
 
   cc @cloud-fan, @gatorsmile, thanks for your reviewing in advance.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] HyukjinKwon closed pull request #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases

Posted by GitBox <gi...@apache.org>.
HyukjinKwon closed pull request #27510: [SPARK-30592][SQL][FOLLOWUP] Add some round-trip test cases
URL: https://github.com/apache/spark/pull/27510
 
 
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org