Peter Lamut

Results 8 comments of Peter Lamut

I added basic JSON schema support (alpha) - https://github.com/plamut/colander/commit/5385f02603c6086fb32662a57ecaa60d5be8080d However, this is work is far from finished, there are several things that deserve a closer look: - JSON schema specification...

Hey, sorry for the late reply, I'm overwhelmed with work. I agree with the proposition, to_json can indeed be renamed to to_json_schema, it's more explicit on what it actually do....

@software-dov The `return_immediately` argument should not be set to `True` ever, as that can cause users to have more trouble retrieving the messages even if they are available, they might...

As per discussion, we will not modify the templates for now, but instead add the change through `synth.py`. We can keep this open as a possible low priority feature request...

BigQuery client library maintainer here. Behind the scenes, `to_dataframe()` [streams](https://github.com/googleapis/python-bigquery/blob/790d11bdbbf50d2f30c09ba4660d420ba2212866/google/cloud/bigquery/_pandas_helpers.py#L616-L740) table data from the server using a thread pool executor in the background. Each thread executes the [_download_table_bqstorage_stream()](https://github.com/googleapis/python-bigquery/blob/790d11bdbbf50d2f30c09ba4660d420ba2212866/google/cloud/bigquery/_pandas_helpers.py#L590-L599), which converts...

_Unofficial_ answer - this will not be added until Python 2.7 support is dropped, which will only start happening in the client libraries after the last official Python 2.7.x release...

@jamierpond I no longer maintain these libraries, thus cannot really say, sorry. @busunkim96 Do you perhaps know, if this has been considered yet?

I did a quick test and queried some public data, fetching the results as dataframe using BQ Storage API under the hood. The execution flow reached [_ArrowStreamParser.to_arrow()](https://github.com/googleapis/python-bigquery-storage/blob/f7fc4ec45e62739139bf865497ce2e5fbd948d31/google/cloud/bigquery_storage_v1/reader.py#L686-L687), but did not...