Skip to content

Handle files over 2GB#1065

Merged
mziccard merged 2 commits intogoogleapis:gcs-niofrom
jean-philippe-martin:no_eom
Jun 21, 2016
Merged

Handle files over 2GB#1065
mziccard merged 2 commits intogoogleapis:gcs-niofrom
jean-philippe-martin:no_eom

Conversation

@jean-philippe-martin
Copy link

@jean-philippe-martin jean-philippe-martin commented Jun 20, 2016

ParallelCountBytes fails for files over 2GB. It turns out that's because the position is stored internally as an int instead of a long in gcloud-java-storage. This CL fixes this. Once we merge into master, it'll solve the same problem for master.

cc: @jart

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 20, 2016
.setIfMetagenerationNotMatch(IF_METAGENERATION_NOT_MATCH.getLong(options))
.setIfGenerationMatch(IF_GENERATION_MATCH.getLong(options))
.setIfGenerationNotMatch(IF_GENERATION_NOT_MATCH.getLong(options));
if (position<0) {

This comment was marked as spam.

This comment was marked as spam.

@mziccard
Copy link
Contributor

Nice catch, see one minor comment. I wonder whether we should merge this fix into master so that we don't have to wait for the bigger gcs-nio to be merged in.

@jean-philippe-martin
Copy link
Author

Tests are green. Are we OK to merge this pull request?

@mziccard mziccard merged commit 576c763 into googleapis:gcs-nio Jun 21, 2016
@jean-philippe-martin
Copy link
Author

Thank you!

@jean-philippe-martin jean-philippe-martin deleted the no_eom branch June 21, 2016 17:39
mziccard pushed a commit to mziccard/gcloud-java that referenced this pull request Jun 29, 2016
* Handle files over 2GB

* Use Guava's checkArgument
mziccard pushed a commit to mziccard/gcloud-java that referenced this pull request Jun 30, 2016
* Handle files over 2GB

* Use Guava's checkArgument
suztomo pushed a commit that referenced this pull request Feb 1, 2023
#1065)

* chore: copy functionality from io.grpc.internal.SharedResourceHolder

Included related classes and test file. Tests were adapted to use an
easymock mock

* fix: remove grpc-core from deps

* fix: add used undeclared dependency jsr305

* fix: add copy explanation comments, remove `@ThreadSafe` and related dependency

* Update google-cloud-core-grpc/src/main/java/com/google/cloud/grpc/LogExceptionRunnable.java

Co-authored-by: Mike Eltsufin <meltsufin@google.com>

* Update google-cloud-core-grpc/src/main/java/com/google/cloud/grpc/SharedResourceHolder.java

Co-authored-by: Mike Eltsufin <meltsufin@google.com>

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: Mike Eltsufin <meltsufin@google.com>
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
meltsufin pushed a commit that referenced this pull request Dec 22, 2025
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
chingor13 pushed a commit that referenced this pull request Jan 22, 2026
…14.0 (#1065)

* chore(deps): update dependency com.google.cloud:libraries-bom to v26.14.0

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
chingor13 pushed a commit that referenced this pull request Feb 24, 2026
)

[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [org.mockito:mockito-core](https://togithub.com/mockito/mockito) | `4.8.0` -> `4.8.1` | [![age](https://badges.renovateapi.com/packages/maven/org.mockito:mockito-core/4.8.1/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/org.mockito:mockito-core/4.8.1/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/org.mockito:mockito-core/4.8.1/compatibility-slim/4.8.0)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/org.mockito:mockito-core/4.8.1/confidence-slim/4.8.0)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>mockito/mockito</summary>

### [`v4.8.1`](https://togithub.com/mockito/mockito/releases/tag/v4.8.1)

[Compare Source](https://togithub.com/mockito/mockito/compare/v4.8.0...v4.8.1)

<sup><sup>*Changelog generated by [Shipkit Changelog Gradle Plugin](https://togithub.com/shipkit/shipkit-changelog)*</sup></sup>

##### 4.8.1

-   2022-10-17 - [6 commit(s)](https://togithub.com/mockito/mockito/compare/v4.8.0...v4.8.1) by andrepaschoal, dependabot\[bot]
-   Possible fix [#&#8203;2765](https://togithub.com/mockito/mockito/issues/2765): Add task to download package-list file from java as element-list [(#&#8203;2766)](https://togithub.com/mockito/mockito/pull/2766)
-   JavaDoc warning is blocking all pull requests [(#&#8203;2765)](https://togithub.com/mockito/mockito/issues/2765)
-   Bump versions.junitJupiter from 5.9.0 to 5.9.1 [(#&#8203;2758)](https://togithub.com/mockito/mockito/pull/2758)
-   Bump groovy from 3.0.12 to 3.0.13 [(#&#8203;2756)](https://togithub.com/mockito/mockito/pull/2756)
-   Bump com.diffplug.spotless from 6.10.0 to 6.11.0 [(#&#8203;2753)](https://togithub.com/mockito/mockito/pull/2753)
-   Bump org.eclipse.osgi from 3.18.0 to 3.18.100 [(#&#8203;2751)](https://togithub.com/mockito/mockito/pull/2751)
-   Bump versions.bytebuddy from 1.12.14 to 1.12.16 [(#&#8203;2747)](https://togithub.com/mockito/mockito/pull/2747)

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, click this checkbox.

---

This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/google-auth-library-java).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzMi4yNDEuNyIsInVwZGF0ZWRJblZlciI6IjMyLjI0MS4xMSJ9-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants