Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Add Test runs for Python 3.7 and remove 3.4 - #5295

#5295
Merged
crwilcox merged 3 commits into
googleapis:mastergoogleapis/google-cloud-python:masterfrom
crwilcox:drop-34-add-37crwilcox/google-cloud-python:drop-34-add-37Copy head branch name to clipboard
May 4, 2018
Merged

Add Test runs for Python 3.7 and remove 3.4#5295
crwilcox merged 3 commits into
googleapis:mastergoogleapis/google-cloud-python:masterfrom
crwilcox:drop-34-add-37crwilcox/google-cloud-python:drop-34-add-37Copy head branch name to clipboard

Conversation

@crwilcox

@crwilcox crwilcox commented May 3, 2018

Copy link
Copy Markdown
Contributor

There are very few people using 3.4 and 3.7 is coming up. A few of the packages cannot use 3.7 yet

PubSub: #5292
Monitoring: #5293
BigQuery: #5294

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label May 3, 2018
tswast
tswast approved these changes May 3, 2018
View reviewed changes

@tswast tswast left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice

@tseaver

tseaver commented May 3, 2018

Copy link
Copy Markdown
Contributor

Are you sure about the 3.4 bit? It is still the default python3 for LTS Linux versions, AFAIK.

@crwilcox

crwilcox commented May 3, 2018 •
edited by theacodes
Loading

Copy link
Copy Markdown
Contributor Author

@tseaver: @jonparrott looked up the stats and said ~4%. We can leave it if you have strong feelings about it. It seemed a convenient time to not elongate the test runs is all.

@theacodes

theacodes commented May 3, 2018 via email

Copy link
Copy Markdown
Contributor

@theacodes

theacodes commented May 3, 2018

Copy link
Copy Markdown
Contributor

Okay, looking over the last 6 months of download stats from BigQuery:


  SELECT
    details.python, COUNT(*) as total_downloads,
  FROM
    TABLE_DATE_RANGE(
      [the-psf:pypi.downloads],
      DATE_ADD(CURRENT_TIMESTAMP(), -6, "month"),
      CURRENT_TIMESTAMP()
    )
  WHERE
    file.project LIKE 'google-cloud%'
  GROUP BY details.python
  ORDER BY details.python DESC

Our packages have been installed a total of 54,568,855 times with 785,977 of those downloads originating from a Python 3.4 version. That means that Python 3.4 accounts for just 1.44% of our install base. By comparison, 3.4 accounts for 3.58% of the overall installs for all packages over the last 6 months.

Summoning @tswast to make sure I didn't do something stupid here.

@tswast

tswast commented May 3, 2018

Copy link
Copy Markdown
Contributor

Query looks right, though I'd filter out the "bandersnatch" installs with details.installer.name = 'pip' if I were you. In my experiments those are a significant percentage of downloads and don't actually count as installs since they mirror all of PyPI.

P.S. I could give you some help updating to standard SQL syntax if you'd like.

@theacodes

theacodes commented May 3, 2018

Copy link
Copy Markdown
Contributor

Thanks @tswast. Limiting to just pip installs made the Py 3.4 percentage even lower at 1.43%. ;)

@crwilcox
crwilcox merged commit 10cd5e5 into googleapis:master May 4, 2018
@tswast tswast mentioned this pull request May 29, 2018
Merged
@tswast

tswast commented May 29, 2018

Copy link
Copy Markdown
Contributor

As pointed out by @tseaver in #5401, this also requires an update to all setup.py files.

@theacodes theacodes mentioned this pull request Jul 25, 2018
Merged
atulep pushed a commit that referenced this pull request Apr 3, 2023
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
cddc37b 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
atulep pushed a commit that referenced this pull request Apr 18, 2023
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
ee130ea 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
parthea pushed a commit that referenced this pull request Jun 4, 2023
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
57002c6 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
parthea pushed a commit that referenced this pull request Jun 4, 2023
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
fef06cb 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
parthea pushed a commit that referenced this pull request Jul 6, 2023
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
ac2eee4 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
parthea pushed a commit that referenced this pull request Sep 22, 2023
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
6311be8 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
parthea pushed a commit that referenced this pull request Sep 22, 2023
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
03f7ef6 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
parthea pushed a commit that referenced this pull request Sep 22, 2023
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
a7830f4 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
parthea pushed a commit that referenced this pull request Sep 22, 2023
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
600638a 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
parthea pushed a commit that referenced this pull request Sep 22, 2023
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
83f633b 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
parthea pushed a commit that referenced this pull request Sep 22, 2023
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
8e6a031 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
parthea pushed a commit that referenced this pull request Oct 21, 2023
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
6d5c710 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
parthea pushed a commit that referenced this pull request Oct 21, 2023
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
4a3bfd9 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
parthea pushed a commit that referenced this pull request Oct 21, 2023
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
a66fd7d 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
parthea pushed a commit that referenced this pull request Oct 22, 2023
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
2a9cfd8 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
parthea pushed a commit that referenced this pull request Oct 22, 2023
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
cb20611 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
parthea pushed a commit that referenced this pull request Oct 31, 2023
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
6877b31 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
parthea pushed a commit that referenced this pull request Nov 22, 2025
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
1017882 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
parthea pushed a commit that referenced this pull request Nov 24, 2025
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
2198ab4 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
parthea pushed a commit that referenced this pull request Nov 24, 2025
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
1f9e8e8 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
parthea pushed a commit that referenced this pull request Nov 24, 2025
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
5ad8f93 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
parthea pushed a commit that referenced this pull request Nov 24, 2025
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
eee9ac6 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
parthea pushed a commit that referenced this pull request Nov 24, 2025
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
ea52357 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
parthea pushed a commit that referenced this pull request Nov 24, 2025
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
c4365d7 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
parthea pushed a commit that referenced this pull request Nov 24, 2025
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
87a741a 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
parthea pushed a commit that referenced this pull request Nov 24, 2025
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
437a139 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
parthea pushed a commit that referenced this pull request Mar 2, 2026
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
065d34b 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
parthea pushed a commit that referenced this pull request Mar 9, 2026
@crwilcox
Add Test runs for Python 3.7 and remove 3.4 (#5295)
e252c45 
* remove 3.4 from unit test runs

* add 3.7 to most packages. PubSub, Monitoring, BigQuery not enabled

* Fix #5292 by draining queue in a way compatible with SimpleQueue and Queue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tswast tswast tswast approved these changes
@dhermes dhermes Awaiting requested review from dhermes
@liyanhui1228 liyanhui1228 Awaiting requested review from liyanhui1228
@lukesneeringer lukesneeringer Awaiting requested review from lukesneeringer
@tseaver tseaver Awaiting requested review from tseaver
@waprin waprin Awaiting requested review from waprin
+1 more reviewer
@theacodes theacodes theacodes approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@crwilcox @tseaver @theacodes @tswast @googlebot
Morty Proxy This is a proxified and sanitized view of the page, visit original site.