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 2022/06/23 02:16:56 UTC
[GitHub] [spark] HyukjinKwon commented on pull request #36957: [SPARK-39555][PYTHON] Make createTable and listTables in the python side support 3-layer-namespace
HyukjinKwon commented on PR #36957:
URL: https://github.com/apache/spark/pull/36957#issuecomment-1163848082
That's sort of annoying problem yeah. We should build Spark w/ `sbt test:package`, and then run the tests. Should probably have a logic like https://github.com/apache/spark/blob/master/python/pyspark/sql/tests/test_dataframe.py#L1204-L1238. Another option is to move the test codes into somewhere within the main code with `private[sql]`. the classes or methods with `private[sql]` can be accessed via Py4J (because that syntax is specific to Scala, and Java class files don't understand them).
--
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@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org