Skip to content

[Task]: Update BigQueryIO.setTriggeringFrequency documentation #23949

Closed

Description

@johnjcasey

What needs to happen?

BigQueryIO currently documents triggering frequency as:

This is only applicable when the write method is set to {@link Method#FILE_LOADS}, and only when writing an unbounded {@link PCollection}.

However, we use triggering frequency for STORAGE_WRITE_API and STORAGE_API_AT_LEAST_ONCE as well. This documentation should be updated to reflect this.

In addition, during the validation component of BigQueryIO.write().expand(), we check for unbounded && (file loads || storage write api) to see if a triggering frequency is set. This should probably also be checked if storage write at least once is set.

Issue Priority

Priority: 2

Issue Component

Component: io-java-gcp

Activity

  1. DhirajSardar commented on Dec 3, 2022

    @DhirajSardar

    I would like to contribute . Can anyone help me ? I am a newbie.

  2. johnjcasey commented on Dec 5, 2022

    @johnjcasey
    ContributorAuthor

    For sure! https://beam.apache.org/contribute/ here is the contribution guide. Take a quick read through that, and then fork the repo to get started

  3. nikhilpothuru commented on Mar 27, 2023

    @nikhilpothuru

    .take-issue

  4. nikhilpothuru commented on Mar 27, 2023

    @nikhilpothuru

    @johnjcasey Hi John, I was reading through the guidelines for joining the ASF slack workspace and it looks like non-ASF members (those without an @apache.org address) need to be invited. I was wondering if I could join the workspace. My email is as follows: pothurunikhil@gmail.com

  5. Vivek-2004 commented on Apr 8, 2023

    @Vivek-2004

    I want to work on this issue .

  6. 11 remaining items

  7. rajkgupt commented on May 20, 2025

    @rajkgupt
    Contributor

    .take-issue

  8. Arunodoy18 commented on Nov 22, 2025

    @Arunodoy18
  9. tarun-google commented on Dec 12, 2025

    @tarun-google
    Contributor

    I was going through BigQueryIO. looks like the triggering frequency is actually ignored when the method is STORAGE_API_AT_LEAST_ONCE. So, i do not see any issue

    1. The validation throws the warning that triggering frequency is ignored
    2. Inconsistent writes doe not create groups based on triggering frequency while
  10. Vicky-8223 commented on Dec 31, 2025

    @Vicky-8223

    I would like to contribute on this issue.I am new contributer and would like to take this up.
    .take-issue

  11. shivanshjais22 commented on Jan 26, 2026

    @shivanshjais22

    Hi, I’d like to work on this issue. Could you please assign it to me?

  12. KartikeyTripathi469 commented on Mar 16, 2026

    @KartikeyTripathi469

    Hello, I would like to work on this issue.
    Could you please assign it to me?

  13. Subramanya-Veeregowda commented on Apr 1, 2026

    @Subramanya-Veeregowda
    Contributor

    Hi, I noticed there is already an open PR (#37205) addressing this issue.

    I went through the discussion and would like to help if there are any pending changes or gaps that still need to be addressed. Please let me know if I can contribute.

    Thanks!

  14. Subramanya-Veeregowda commented on Apr 18, 2026

    @Subramanya-Veeregowda
    Contributor

    Hi, I noticed this issue and PR(#37205 ) has been inactive for a while and has some conflicts.

    I’d like to help move this forward. I will open a new one with a minimal fix (only updating the relevant Javadoc)?

    @ahmedabu98 if there is any specific requirements then mention it ready to work on it.

  15. ahmedabu98 commented on May 7, 2026

    @ahmedabu98
    Contributor

    I’d like to help move this forward. I will open a new one with a minimal fix (only updating the relevant Javadoc)?

    Hey @Subramanya-Veeregowda, yes that should be enough to get it in!

  16. damccorm commented on Aug 31, 2026

    @damccorm
    Contributor

    I was going through BigQueryIO. looks like the triggering frequency is actually ignored when the method is STORAGE_API_AT_LEAST_ONCE. So, i do not see any issue

    1. The validation throws the warning that triggering frequency is ignored
    2. Inconsistent writes doe not create groups based on triggering frequency while

    @tarun-google should the issue be closed then?

  17. tarun-google commented on Sep 2, 2026

    @tarun-google
    Contributor

    Yes, we can close this issue. I asked AI to cross validate since its been a while

    Notes:

    1. STORAGE_API_AT_LEAST_ONCE intentionally does not use triggeringFrequency—it routes to expandInconsistent() which streams writes directly without batching or buffering delay, and BigQueryIO.expand() explicitly disallows/ignores triggering frequency for this method.
    2. The Javadoc update for STORAGE_WRITE_API was already merged in Update triggering frequency doc #30457 (which clarified that withTriggeringFrequency applies to FILE_LOADS and STORAGE_WRITE_API).
  18. added this to the 2.77.0 Release milestone on Sep 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Type

No type

Projects

No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions