Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix pyarrow Table type hinting in cursor.py #1

Closed
wants to merge 318 commits into from

Conversation

thomasaarholt
Copy link
Owner

Please answer these questions before submitting your pull requests. Thanks!

  1. What GitHub issue is this PR addressing? Make sure that there is an accompanying issue to your PR.

    Fixes #NNNN

  2. Fill out the following pre-review checklist:

    • I am adding a new automated test(s) to verify correctness of my new code
    • I am adding new logging messages
    • I am modifying authorization mechanisms
    • I am adding new credentials
    • I am modifying OCSP code
    • I am adding a new dependency
  3. Please describe how your code solves the related issue.

    Please write a short description of how your code change solves the related issue.

sfc-gh-jdu and others added 30 commits August 25, 2022 15:10
…edb#1238)

* SNOW-641540 Fix dtypes of fetch_pandas_all for empty result (snowflakedb#1226)

* SNOW-641540 Fix dtypes of fetch_pandas_all for empty result

When the result is empty, the pandas dataframe information is lost
when creating the iterator. To keep the information, we store the
to_pandas result, and use it for empty dataframe case.

* Address comment

* use list

* format

* Add DESCRIPTION
… chunk rowcount (snowflakedb#1249)

* Attempt to add INFO level logging message regarding first chunk rowcount
* Added test for logging results in first chunk
* Moved log tests to test_insert_select, test_insert_and_select_by_separate_connection, and test_fetchmany
…wflakedb#1252)

* Exposed to_parquet kwargs parameter in write_pandas
* Added tests for kwargs and explicit type annotation
* Reduced parametrization for tests
* SNOW-669945 Bump cryptography version to <39.0.0

Description

This version bump is to avoid version conflict with pyOpenSSL

Testing

Added a new workflow test to run pip-compile

* setup python

* move test to tox and merge pip install in fips

* Add merge gate using tox
* Fix special case for chunk helper

* Address PR comment

* Update src/snowflake/connector/pandas_tools.py

Address PR comments

Co-authored-by: Mark Keller <[email protected]>

* Address PR comments

Co-authored-by: Mark Keller <[email protected]>
…ion (snowflakedb#1273)

* Parallelize closing check

* Use reversed insertion order

* Fix tests

* Fix tests

* Address PR comments and add test

* Change value type to None

* Address PR comments

* Break after first positive and attempt to cancel future execution
…snowflakedb#1288)

* SNOW-680774 Check CLIENT_LOG_DIR_PATH_DOCKER for ENABLE_TELEMETRY_LOG

Description

test_util is supposed to only be used by snowflake regression test,
but currently it runs for all Jenkins linux hosts, which is not
desired. A quick fix is to also check CLIENT_LOG_DIR_PATH_DOCKER,
which presumbly is not used by most customers if not all.

Testing

existing tests

* fix fix_lint

* address comment

Co-authored-by: Mark Keller <[email protected]>
…8.1 (snowflakedb#1303)

* SNOW-683718: Bumped up PythonConnector PATCH version from 2.8.0 to 2.8.1
* Update Jenkinsfile

* Update Jenkinsfile
sfc-gh-aling and others added 28 commits October 20, 2023 22:11
* Fix urllib bug in vendored library

* changelog updates
* SNOW-892716 Improve localfs put/get test

Description

This is a long overdue improvement from early slack message from
Brandon

Testing

unit test

* comment
…edb#1787)

Co-authored-by: Jenkins User <snowflake-provisioner>
Co-authored-by: Kiran Dama <[email protected]>
Co-authored-by: github-actions <[email protected]>
* updating vendored requests to 2.31.0

* updating vendored urllib3 to 1.26.18

* putting back urllib3 pin

* adding changelog entry

---------

Co-authored-by: Sophie Tan <[email protected]>
…edb#1797)

* Preparation for 3.4.1 release

* Update requirements files

* Add version change

* Change release date to tomorrow

---------

Co-authored-by: github-actions <[email protected]>
Co-authored-by: Benny Lu <[email protected]>
Co-authored-by: Benny Lu <[email protected]>
…flakedb#1719)

Updates the requirements on [pandas](https://github.com/pandas-dev/pandas) to permit the latest version.
- [Release notes](https://github.com/pandas-dev/pandas/releases)
- [Commits](pandas-dev/pandas@v1.0.0...v2.1.0)

---
updated-dependencies:
- dependency-name: pandas
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Shixuan Fan <[email protected]>
@github-actions github-actions bot locked and limited conversation to collaborators Mar 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.