Sujit Madiraju

Results 35 comments of Sujit Madiraju

I had a similar issue and was able to resolve it recently. Just to clarify, are you sure the listener is being stopped before you hit "Ask" to start it...

Hi @mbasmanova, I would like to claim the `json_parse` function.

Thanks @mbasmanova, must've missed that. Then may I claim the`flatten` array function? Doesn't look like it's been added yet.

Hi @mbasmanova, I would actually like to take the `split_to_map` string function first instead if that's alright.

Hi @mbasmanova, I would also like to claim the following functions: ``` from_iso8601_date from_iso8601_timestamp current_timezone ```

@majetideepak Thanks for pointing that out, I recently made the same observation myself. Corrected behavior to comply with Presto Java.

@majetideepak @aditi-pandit please take a look, changes have been finalized.

@mbasmanova Thanks for the review. I was told to break up the all-encompassing PR into separate PRs for each function signature, for ease of review, as each set of changes...

@mbasmanova Sorry for the confusion - I think I better understand your point now about making `toIso8601()` a static function. Though I did want to mention, there is an existing...