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/04/23 18:53:08 UTC

[GitHub] [spark] nucflash commented on pull request #25193: [SPARK-28439][PYTHON][SQL] Add support for count: Column in array_repeat

nucflash commented on pull request #25193:
URL: https://github.com/apache/spark/pull/25193#issuecomment-618589980


   I see your point @zero323. I ran into this issue when I passed a column which was the results of a count aggregation and was typed as `bigint` instead of `int`. Explicitly casting columns returned from native functions feels a bit awkward as it hints that the internals are not compatible with each other. But the issue may not lie in `array_repeat` but in `count()`, that I don't know what is best to return, `int` or `bigint`.


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



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