Mailing-List: contact reviews-help@spark.apache.org; run by ezmlm
Precedence: bulk
From: davies <git@git.apache.org>
To: reviews@spark.apache.org
Reply-To: reviews@spark.apache.org
References: <git-pr-13571-spark@git.apache.org>
In-Reply-To: <git-pr-13571-spark@git.apache.org>
Subject: [GitHub] spark issue #13571: [SPARK-15369][WIP][RFC][PySpark][SQL] Expose potential t...
Content-Type: text/plain
Message-Id: <20161012203709.9B13DE0061@git1-us-west.apache.org>
Date: Wed, 12 Oct 2016 20:37:09 +0000 (UTC)
archived-at: Wed, 12 Oct 2016 20:37:12 -0000

Github user davies commented on the issue:

    https://github.com/apache/spark/pull/13571
  
    Thanks for the prototype and sending the PR out, this looks interesting.
    
    I don't know how mature Jython currently is, never heard a company who use it. The last release took to years from beta to release, maybe that's a signal that the community behind Jython is not that active.
    
    The license Jython use is unique, I don't know whether it is OK to package with Spark or not. Also the standalone jar is 37M, that's pretty big for a experimental feature.
    
    This PR also introduce a public API, even we mark it as experimental, it still require some effort to maintain it even deprecate and remove it. I'm sure it's worth or not.
    
    We could leave the JIRA and PR open to gather more feedback, it could be useful in case that some people want to try it out.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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