Gabriel

Results 53 comments of Gabriel

Thank you, I understand how to use pylint's own `# pylint: disable=` to suppress pylint error messages. I meant the issue as more of a feature-request/enhancement. Personally I think not...

I like it. I think adding a `connect=True/False` keyword argument could also be a good idea. This is how `pymongo` behaves. Although they default to `connect=True`. https://pymongo.readthedocs.io/en/stable/api/pymongo/mongo_client.html#pymongo.mongo_client.MongoClient ```python MongoClient(host='localhost', port=27017,...

Is is possible the have requests that wait indefinitely if the `INVOCATION_TIMEOUT_SECONDS` is not set? `queue.take()` or `map.lock("foo", 200)` Has this been implemented, should this issue be closed?

A related point, I do think it would be very helpful to explicitly enumerate the client config options instead of accepting everything as `**kwargs`. Or at the very least declare...

~I moved renamed `DEFAULT_BASE_URL` -> `CLOUD_BASE_URL` and move it to the `core/http.py`~ An alternative would be to create a ~`great_expectations/constants.py`~ or `great_expectations/data_context/cloud_constants.py` file. https://github.com/great-expectations/great_expectations/pull/6176/commits/5374b7a19f615d6a25396be8acd56a0d0c834553

I have a similar use case with serving very small image files that are pulled from a database. I don't want to have to write to the file system so...

https://gist.github.com/cdkini/90868011502c385da8ce231c2b282bb0

A `Config.factory()`, or `Config.from_dict()` method might express the intent of this method better.

``` Python 3.6.6 (default, Sep 12 2018, 18:26:19) [GCC 8.0.1 20180414 (experimental) [trunk revision 259383]] on linux Type "help", "copyright", "credits" or "license" for more information. ``` ```python >>> import...