panbingkun

Results 183 comments of panbingkun

https://github.com/aws/aws-sdk-java/blob/master/aws-java-sdk-kinesis/src/main/java/com/amazonaws/services/kinesis/AmazonKinesisClient.java#L174~L189

Maybe I need an Amazon account, but I really don't have one at present. Who can help me?

> @panbingkun Let's re-enable the doctest > > https://github.com/apache/spark/blob/514ecc6fc183d7222b9dc299af4df328c71966d1/python/pyspark/sql/functions/builtin.py#L15053-L15058 > > and remove the TODO > > https://github.com/apache/spark/blob/514ecc6fc183d7222b9dc299af4df328c71966d1/python/pyspark/sql/functions/builtin.py#L15010-L15013 > > also cc @HyukjinKwon Okay, done.

> Can the `from_csv` function parse the output of `to_csv()` back? Could you add tests or modify existing tests, please. For this case, currently the `from_csv` function cann't parse the...

> 1. Does CSV source support this? I think it's disallowed > 2. If we want to support this it should better be able to read it back. Problem is...

> I think we should simply disallow this. We could have a legacy conf, but it doesn't much make sense. What does it return for `ArrayType` and `MapType`? Okay, I...

> Would you please add `[PYTHON]` to title considering it has user-facing python change? Thanks! Sure, done. 😄

> Is this considered a bug fix? Do we need to backport it to branch-3.5/3.4 Yea, I think we need to backport it to `branch-3.5/3.4`.

friendly ping @HyukjinKwon, When you are not busy, can you please continue to help review this PR?