You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@sedona.apache.org by "Harjit Singh (Jira)" <ji...@apache.org> on 2022/03/29 02:18:00 UTC

[jira] [Commented] (SEDONA-25) Change Scala Seq type in Adapter

    [ https://issues.apache.org/jira/browse/SEDONA-25?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17513765#comment-17513765 ] 

Harjit Singh commented on SEDONA-25:
------------------------------------

What kind of confusion are we talking about because. Seq is the equivalent of List in Java. Could you please elaborate?

> Change Scala Seq type in Adapter
> --------------------------------
>
>                 Key: SEDONA-25
>                 URL: https://issues.apache.org/jira/browse/SEDONA-25
>             Project: Apache Sedona
>          Issue Type: Task
>            Reporter: Jia Yu
>            Assignee: Jia Yu
>            Priority: Major
>
> Currently, the adapter function requires Seq type list for input fieldNames. This leads to lots of confusion in Scala and Java users: https://github.com/apache/incubator-sedona/blob/master/sql/src/main/scala/org/apache/sedona/sql/utils/Adapter.scala#L134
>  
> To fix that, we'd better use List or Java list as the type.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)