You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@livy.apache.org by GitBox <gi...@apache.org> on 2021/10/22 12:12:01 UTC

[GitHub] [incubator-livy] dohongdayi opened a new pull request #332: Preserve order branch

dohongdayi opened a new pull request #332:
URL: https://github.com/apache/incubator-livy/pull/332


   ## What changes were proposed in this pull request?
   
   Currently the fields in table magic are sorted by field name by default, but in many scenarios, we want to preserve fields of table magic in query order, for example:
   
   `val result = spark.sql("select b, a from table")`
   `%table result`
   
   currently Livy responses json with fields in the order of "a", "b", but we want them in the order of "b", "a" which is consistent with the SQL
   
   ## How was this patch tested?
   
   existing tests & new tests in ScalaInterpreterSpec.
   


-- 
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.

To unsubscribe, e-mail: reviews-unsubscribe@livy.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-livy] dohongdayi commented on pull request #332: [LIVY-871] Add client config to preserve fields of table magic in query order

Posted by GitBox <gi...@apache.org>.
dohongdayi commented on pull request #332:
URL: https://github.com/apache/incubator-livy/pull/332#issuecomment-949573786


   ok to test


-- 
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.

To unsubscribe, e-mail: reviews-unsubscribe@livy.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org