Robert (Bobby) Evans

If I try to parse something with an array as the top level I get an illegal memory access error. ``` scala> Seq("""{"a":100}""","""[{"b":200}]""").toDF("json").repartition(1).selectExpr("*", "get_json_object(json, '$.a') as a", "get_json_object(json, '$.b') as...

I think you already know this also, but I found that I could not have two paths that were ambiguous in the same call. I could not have one that...

On a happy note I was able to get a 1.6x speedup on one of my queries with this patch compared to the version in spark-rapids-jni. But I did have...

I forgot to add that the APIs to take a raw pointer is far from ideal. It might be nice to have a separate library that is compiled against jcuda...

I don't think we want to try and support having anything but a String as the key for a map. At least not until we have a customer use case...

After https://github.com/NVIDIA/spark-rapids/pull/11642 this no longer throws an exception. Instead it falls back to the CPU.

json_tuple was not updated to use the new get_json_object kernel

In my performance tests I see between a 3x and 6x reduction in performance compared to the old implementation, but it does do the right thing most of the time,...

> Wow, so it is actually quite fast. Not sure if I tested it right. A bit of feedback on the quick test. 1. Looks like you only ran it...

We do not want to implement a kernel just for Africa/Casablanca unless we implement it for all time zones!!! https://github.com/NVIDIA/spark-rapids/issues/6839 We picked Africa/Casablanca as an arbitrary time zone. It is...