Skip to content

Conversation

@jbonofre
Copy link
Member

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

  • Make sure the PR title is formatted like:
    [BEAM-<Jira issue #>] Description of pull request
  • Make sure tests pass via mvn clean verify. (Even better, enable
    Travis-CI on your fork and ensure the whole test matrix passes).
  • Replace <Jira issue #> in the title with the actual Jira issue
    number, if there is one.
  • If this contribution is large, please file an Apache
    Individual Contributor License Agreement.

Fixing Javadoc issue on Flink runner (core and examples) allowing to run a mvn clean install -Prelease.

@jbonofre
Copy link
Member Author

Updating the PR to fix java8 examples javadoc as well.

@jbonofre jbonofre force-pushed the BEAM-242-JAVADOC branch from ecc9683 to a7ab48f Compare May 25, 2016 13:20
@jbonofre
Copy link
Member Author

Rebased and fix javadoc on java 8 examples.

@kennknowles
Copy link
Member

If you rebased, then the javadoc building is disabled in the build, so perhaps you can re-enable it for this PR?

@jbonofre
Copy link
Member Author

Good point @kennknowles let me re-enable javadoc. Thanks !

@jbonofre jbonofre force-pushed the BEAM-242-JAVADOC branch from a7ab48f to 6b7d535 Compare May 25, 2016 20:05
@jbonofre
Copy link
Member Author

Updated to re-enable javadoc.

/**
* Run a batch pipeline.
*
* @param args The main arguments.
Copy link
Member

Choose a reason for hiding this comment

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

Also for @param I don't think it is that helpful to require the annotations. Usually it is better to have sentences that actually describe what is going on. Code review is enough to make sure that when @param is needed that someone will add it, but that is maybe 10% of the time or less IMO.

@kennknowles
Copy link
Member

I like most of the changes, but there are some @return and @param lines that just say the same thing that the description says. We should fix the configuration so it doesn't fail in these situations. It really shouldn't - the SDK already does not include these all the time. So where is the configuration reset?

@jbonofre
Copy link
Member Author

I just tried to use:


  <plugin>
    <groupId>org.apache.maven.plugins</groupId>
    <artifactId>maven-javadoc-plugin</artifactId>
    <configuration>
      <additionalparam>-Xdoclint:missing</additionalparam>
    </configuration>
  </plugin>

and it generates good output. Instead of completely disabling the javadoc plugin execution, or even changing the source, I would propose this configuration (at least for the first release).

@jbonofre
Copy link
Member Author

Alternative in PR #403

@davorbonaci
Copy link
Member

#403 merged; this PR is now obsolete.

@jbonofre
Copy link
Member Author

jbonofre commented Jun 1, 2016

Alright, I'm closing this one.

@jbonofre jbonofre closed this Jun 1, 2016
@jbonofre jbonofre deleted the BEAM-242-JAVADOC branch June 7, 2016 19:43
dhalperi pushed a commit to dhalperi/beam that referenced this pull request Aug 23, 2016
Backport Beam PR-828: Set Gcs upload buffer size to 1M in DataflowRunner streaming mode
tvalentyn pushed a commit to tvalentyn/beam that referenced this pull request May 15, 2018
damccorm pushed a commit that referenced this pull request Jan 6, 2023
…hange the example (#24731)

* Integration test to load the default example of the default SDK and change the example (#24730) (#24729)

* Fix formatting and README (#24730)

* Support collection v1.17.0 (#24730)

* LoadingIndicator on chaning examples, remove duplicating licenses (#24730)

* Add a missing license header (#24730)

* Integration test for changing SDK and running code (#24779) (#382)

* Integration test for changing SDK and running code (#24779)

* Rename an integration test (#24779)

* Use enum to switch SDK in integration test (#24779)

* Find SDK in a dropdown by key (#24779)

* Add a TODO (#24779)

* Fix exports (#24779)

* Issue24779 integration changing sdk from 24370 (#387)

* Integration test for changing SDK and running code (#24779)

* Rename an integration test (#24779)

* Use enum to switch SDK in integration test (#24779)

* Find SDK in a dropdown by key (#24779)

* Add a TODO (#24779)

* Fix exports (#24779)

* Integration tests miscellaneous UI (#383)

* miscellaneous ui integration tests

* reverted pubspec.lock

* gradle tasks ordered alhpabetically

* integration tests refactoring

* clean code

* integration tests miscellaneous ui fix pr

* rename method

* added layout adaptivity

* A minor cleanup (#24779)

Co-authored-by: Dmitry Repin <mr.malarg@gmail.com>

Co-authored-by: Dmitry Repin <mr.malarg@gmail.com>
pabloem pushed a commit that referenced this pull request Mar 9, 2023
* Integration test to load the default example of the default SDK and change the example (#24730) (#24729)

* Fix formatting and README (#24730)

* Support collection v1.17.0 (#24730)

* LoadingIndicator on chaning examples, remove duplicating licenses (#24730)

* Add a missing license header (#24730)

* Integration test for changing SDK and running code (#24779) (#382)

* Integration test for changing SDK and running code (#24779)

* Rename an integration test (#24779)

* Use enum to switch SDK in integration test (#24779)

* Find SDK in a dropdown by key (#24779)

* Add a TODO (#24779)

* Fix exports (#24779)

* Issue24779 integration changing sdk from 24370 (#387)

* Integration test for changing SDK and running code (#24779)

* Rename an integration test (#24779)

* Use enum to switch SDK in integration test (#24779)

* Find SDK in a dropdown by key (#24779)

* Add a TODO (#24779)

* Fix exports (#24779)

* Integration tests miscellaneous UI (#383)

* miscellaneous ui integration tests

* reverted pubspec.lock

* gradle tasks ordered alhpabetically

* integration tests refactoring

* clean code

* integration tests miscellaneous ui fix pr

* rename method

* added layout adaptivity

* A minor cleanup (#24779)

Co-authored-by: Dmitry Repin <mr.malarg@gmail.com>

* integration tests run and editing

* example selector test

* minor fixes

* rat

* fix pr

* minor

* minor

* rat

* integration test finder written

* integration test minor fixes

* minor fixes

* removed comment

* minor fixes

* playground integration tests minor fixes

* integration test pumpAnSettleNoException

* integration test shortcut refactor

* integration test another changing shortcuts running

* upgrade to flutter 3.7.1

* workaround comment

* playground frontend updated major versions

* issues 25329 25331 25336

* 25329 extract connectivity extension to separate file

* Upgrade Flutter to 3.7.3 in integration tests (#24730)

* Fix integration test (#24730)

* fix cors issue and added mouse scroll to tags

* Upgrade Flutter in Dockerfile (#24720)

* sorting moved to model

* sorting moved to model

* sorting moved to model

* bugs fix

* issue 25278

* fix pr

* quites fix in en.yaml

* Fix not loading default example (#25528)

* fix compile error

* Refactor output tabs, test embedded playground (#25136) (#439)

* Refactor output tabs, test embedded playground (#25136)

* Clean up (#25136)

* Change example paths to IDs in integration tests

* Panning the graph tab (#25118)

---------

Co-authored-by: Alexey Inkin <alexey.inkin@akvelon.com>
Co-authored-by: alexeyinkin <leha@inkin.ru>
pabloem pushed a commit that referenced this pull request Mar 27, 2023
* Integration test to load the default example of the default SDK and change the example (#24730) (#24729)

* Fix formatting and README (#24730)

* Support collection v1.17.0 (#24730)

* LoadingIndicator on chaning examples, remove duplicating licenses (#24730)

* Add a missing license header (#24730)

* Integration test for changing SDK and running code (#24779) (#382)

* Integration test for changing SDK and running code (#24779)

* Rename an integration test (#24779)

* Use enum to switch SDK in integration test (#24779)

* Find SDK in a dropdown by key (#24779)

* Add a TODO (#24779)

* Fix exports (#24779)

* Issue24779 integration changing sdk from 24370 (#387)

* Integration test for changing SDK and running code (#24779)

* Rename an integration test (#24779)

* Use enum to switch SDK in integration test (#24779)

* Find SDK in a dropdown by key (#24779)

* Add a TODO (#24779)

* Fix exports (#24779)

* Integration tests miscellaneous UI (#383)

* miscellaneous ui integration tests

* reverted pubspec.lock

* gradle tasks ordered alhpabetically

* integration tests refactoring

* clean code

* integration tests miscellaneous ui fix pr

* rename method

* added layout adaptivity

* A minor cleanup (#24779)

Co-authored-by: Dmitry Repin <mr.malarg@gmail.com>

* integration tests run and editing

* example selector test

* minor fixes

* rat

* fix pr

* minor

* minor

* rat

* integration test finder written

* integration test minor fixes

* minor fixes

* removed comment

* minor fixes

* playground integration tests minor fixes

* integration test pumpAnSettleNoException

* integration test shortcut refactor

* integration test another changing shortcuts running

* upgrade to flutter 3.7.1

* workaround comment

* playground frontend updated major versions

* issues 25329 25331 25336

* 25329 extract connectivity extension to separate file

* Upgrade Flutter to 3.7.3 in integration tests (#24730)

* Fix integration test (#24730)

* fix cors issue and added mouse scroll to tags

* Upgrade Flutter in Dockerfile (#24720)

* sorting moved to model

* sorting moved to model

* sorting moved to model

* bugs fix

* issue 25278

* fix pr

* quites fix in en.yaml

* Fix not loading default example (#25528)

* fix compile error

* Refactor output tabs, test embedded playground (#25136) (#439)

* Refactor output tabs, test embedded playground (#25136)

* Clean up (#25136)

* Change example paths to IDs in integration tests

* issue25731 shortcut keys sequence

* Update playground/frontend/playground_components/test/src/models/shortcut_test.dart

Co-authored-by: alexeyinkin <leha@inkin.ru>

* issue25731 fix shortcut modal width, fix shortcut meta key text

* issue25731 fix shortcut modal window

* issue25731 pr fix

* issue25731 fix pr

* issue25731 remane file

* fix test

---------

Co-authored-by: Alexey Inkin <alexey.inkin@akvelon.com>
Co-authored-by: alexeyinkin <leha@inkin.ru>
damccorm pushed a commit that referenced this pull request May 1, 2023
* Integration test to load the default example of the default SDK and change the example (#24730) (#24729)

* Fix formatting and README (#24730)

* Support collection v1.17.0 (#24730)

* LoadingIndicator on chaning examples, remove duplicating licenses (#24730)

* Add a missing license header (#24730)

* Integration test for changing SDK and running code (#24779) (#382)

* Integration test for changing SDK and running code (#24779)

* Rename an integration test (#24779)

* Use enum to switch SDK in integration test (#24779)

* Find SDK in a dropdown by key (#24779)

* Add a TODO (#24779)

* Fix exports (#24779)

* Issue24779 integration changing sdk from 24370 (#387)

* Integration test for changing SDK and running code (#24779)

* Rename an integration test (#24779)

* Use enum to switch SDK in integration test (#24779)

* Find SDK in a dropdown by key (#24779)

* Add a TODO (#24779)

* Fix exports (#24779)

* Integration tests miscellaneous UI (#383)

* miscellaneous ui integration tests

* reverted pubspec.lock

* gradle tasks ordered alhpabetically

* integration tests refactoring

* clean code

* integration tests miscellaneous ui fix pr

* rename method

* added layout adaptivity

* A minor cleanup (#24779)

Co-authored-by: Dmitry Repin <mr.malarg@gmail.com>

* integration tests run and editing

* example selector test

* minor fixes

* rat

* fix pr

* minor

* minor

* rat

* integration test finder written

* integration test minor fixes

* minor fixes

* removed comment

* minor fixes

* playground integration tests minor fixes

* integration test pumpAnSettleNoException

* integration test shortcut refactor

* integration test another changing shortcuts running

* upgrade to flutter 3.7.1

* workaround comment

* playground frontend updated major versions

* issues 25329 25331 25336

* 25329 extract connectivity extension to separate file

* Upgrade Flutter to 3.7.3 in integration tests (#24730)

* Fix integration test (#24730)

* fix cors issue and added mouse scroll to tags

* Upgrade Flutter in Dockerfile (#24720)

* sorting moved to model

* sorting moved to model

* sorting moved to model

* bugs fix

* issue 25278

* fix pr

* quites fix in en.yaml

* Fix not loading default example (#25528)

* fix compile error

* Refactor output tabs, test embedded playground (#25136) (#439)

* Refactor output tabs, test embedded playground (#25136)

* Clean up (#25136)

* Change example paths to IDs in integration tests

* issue25640 tob ci

* fix tob ci

* rename ci process

* test add new line to main

* test add new line to main

* commented unit test run

* issue25640 changed server path

* issue25640 tests on welcome page

* deleted config.g.dart

* issue25640 pr fixes

* Update .github/workflows/tour_of_beam_frontend_test.yml

Co-authored-by: alexeyinkin <leha@inkin.ru>

* Update learning/tour-of-beam/frontend/integration_test/welcome_page_test.dart

Co-authored-by: alexeyinkin <leha@inkin.ru>

* Improve tests (#25640)

* issue25640 tour page tests

* pr fix

* removed import

* pr fix

* fix test

* 25640 fixed pubspec.lock

* issue25640 fix readme

* updated readme

* issue25640 fixed after master merge

* issue25483 ToB pipeline options

* removed unnecesary variable

* pr fix

* Update learning/tour-of-beam/frontend/assets/translations/en.yaml

Co-authored-by: alexeyinkin <leha@inkin.ru>

* playground hides when snippet does not exists

* pipeline options extracted to playground components

* issue25483 pipeline options

* added errors handling, fix pr

* refactoring

* Revert "refactoring"

This reverts commit 1540961.

* removed unnecessary constants

* playground controller in tour notifier becomes nullable

* playground controller returned to non nullable in tour notifier

* playground controller actions

* removed unnecessary code

* tob scaffold wrapped with animated builder

* minor fixes

* partially fixed tests

* Upgrade flutter_code_editor to v0.2.19 (#25640)

* Replace output SelectableText with a CodeField instance (#25640)

* Trigger ToB integration tests (#25640)

* Clean up (#25640)

* Enable manual workflow runs for Playground and ToB integration tests (#25640)

---------

Co-authored-by: Alexey Inkin <alexey.inkin@akvelon.com>
Co-authored-by: alexeyinkin <leha@inkin.ru>
lostluck pushed a commit to lostluck/beam that referenced this pull request May 1, 2023
* Integration test to load the default example of the default SDK and change the example (apache#24730) (apache#24729)

* Fix formatting and README (apache#24730)

* Support collection v1.17.0 (apache#24730)

* LoadingIndicator on chaning examples, remove duplicating licenses (apache#24730)

* Add a missing license header (apache#24730)

* Integration test for changing SDK and running code (apache#24779) (apache#382)

* Integration test for changing SDK and running code (apache#24779)

* Rename an integration test (apache#24779)

* Use enum to switch SDK in integration test (apache#24779)

* Find SDK in a dropdown by key (apache#24779)

* Add a TODO (apache#24779)

* Fix exports (apache#24779)

* Issue24779 integration changing sdk from 24370 (apache#387)

* Integration test for changing SDK and running code (apache#24779)

* Rename an integration test (apache#24779)

* Use enum to switch SDK in integration test (apache#24779)

* Find SDK in a dropdown by key (apache#24779)

* Add a TODO (apache#24779)

* Fix exports (apache#24779)

* Integration tests miscellaneous UI (apache#383)

* miscellaneous ui integration tests

* reverted pubspec.lock

* gradle tasks ordered alhpabetically

* integration tests refactoring

* clean code

* integration tests miscellaneous ui fix pr

* rename method

* added layout adaptivity

* A minor cleanup (apache#24779)

Co-authored-by: Dmitry Repin <mr.malarg@gmail.com>

* integration tests run and editing

* example selector test

* minor fixes

* rat

* fix pr

* minor

* minor

* rat

* integration test finder written

* integration test minor fixes

* minor fixes

* removed comment

* minor fixes

* playground integration tests minor fixes

* integration test pumpAnSettleNoException

* integration test shortcut refactor

* integration test another changing shortcuts running

* upgrade to flutter 3.7.1

* workaround comment

* playground frontend updated major versions

* issues 25329 25331 25336

* 25329 extract connectivity extension to separate file

* Upgrade Flutter to 3.7.3 in integration tests (apache#24730)

* Fix integration test (apache#24730)

* fix cors issue and added mouse scroll to tags

* Upgrade Flutter in Dockerfile (apache#24720)

* sorting moved to model

* sorting moved to model

* sorting moved to model

* bugs fix

* issue 25278

* fix pr

* quites fix in en.yaml

* Fix not loading default example (apache#25528)

* fix compile error

* Refactor output tabs, test embedded playground (apache#25136) (apache#439)

* Refactor output tabs, test embedded playground (apache#25136)

* Clean up (apache#25136)

* Change example paths to IDs in integration tests

* issue25640 tob ci

* fix tob ci

* rename ci process

* test add new line to main

* test add new line to main

* commented unit test run

* issue25640 changed server path

* issue25640 tests on welcome page

* deleted config.g.dart

* issue25640 pr fixes

* Update .github/workflows/tour_of_beam_frontend_test.yml

Co-authored-by: alexeyinkin <leha@inkin.ru>

* Update learning/tour-of-beam/frontend/integration_test/welcome_page_test.dart

Co-authored-by: alexeyinkin <leha@inkin.ru>

* Improve tests (apache#25640)

* issue25640 tour page tests

* pr fix

* removed import

* pr fix

* fix test

* 25640 fixed pubspec.lock

* issue25640 fix readme

* updated readme

* issue25640 fixed after master merge

* issue25483 ToB pipeline options

* removed unnecesary variable

* pr fix

* Update learning/tour-of-beam/frontend/assets/translations/en.yaml

Co-authored-by: alexeyinkin <leha@inkin.ru>

* playground hides when snippet does not exists

* pipeline options extracted to playground components

* issue25483 pipeline options

* added errors handling, fix pr

* refactoring

* Revert "refactoring"

This reverts commit 1540961.

* removed unnecessary constants

* playground controller in tour notifier becomes nullable

* playground controller returned to non nullable in tour notifier

* playground controller actions

* removed unnecessary code

* tob scaffold wrapped with animated builder

* minor fixes

* partially fixed tests

* Upgrade flutter_code_editor to v0.2.19 (apache#25640)

* Replace output SelectableText with a CodeField instance (apache#25640)

* Trigger ToB integration tests (apache#25640)

* Clean up (apache#25640)

* Enable manual workflow runs for Playground and ToB integration tests (apache#25640)

---------

Co-authored-by: Alexey Inkin <alexey.inkin@akvelon.com>
Co-authored-by: alexeyinkin <leha@inkin.ru>
lostluck added a commit that referenced this pull request May 2, 2023
* Add channel split separations.

* Use channel leases for parallel processing.

* move data wait to chan + atomic

* Progress and split on execute goroutine.

* Convert metrics tests to short ids.

* Add Provision handler w/ capabilities.

* Add artifact validation to run_rc_validation (#26407)

* Add artifact validation to run_rc_validation

* config file for artifacts

* use java bom contents for validation

* add pip_pre = True

* Update tox.ini

* Remove out of date experimental warning on SDFs (#26450)

* [ToBF] Refinement 26.04.23 (#26428)

* content tree rebuilds on sdk change

* expandable parent node widget

* comment fix

* comment fix

---------

Co-authored-by: darkhan.nausharipov <darkhan.nausharipov@kzn.akvelon.com>

* Allow implicit flattening for yaml inputs. (#26423)

* Add field annotations for high-priority Syndeo schema transforms (#26384)

add changes

* [Go SDK] Timers with new datalayer (#26101)

* added timer package

* add timer changes and merged with rebo's pr

* timer fired in stateful

* error setting new timer in ontimer

* looping timers work

* send fv instead of bytes

* changes to coder/pardo

* works for all cases, only cleanup left

* remove comments and validate onTimer

* generic coder for user key

* fixes coder end to end

* remove logs

* add unit test and refactor

* add docs

* new example

* fix static lint

* support emitters

* allow input col of CoGBK as well

* unit tests, periodic impulse, minor refactor

* update PipelineTimer interface, minor refactor, doc comment for example

* add warn message

* single edge timer coder, rm kv coder check, cache encoder,decoder

* Update chromedriver-binary requirement in /sdks/python

Updates the requirements on [chromedriver-binary](https://github.com/danielkaiser/python-chromedriver-binary) to permit the latest version.
- [Release notes](https://github.com/danielkaiser/python-chromedriver-binary/releases)
- [Commits](danielkaiser/python-chromedriver-binary@v100.0.4896.20.0...v113.0.5672.24.0)

---
updated-dependencies:
- dependency-name: chromedriver-binary
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>

* Automation: Tour of Beam infrastructure deployment (#25793)

* Added Terraform scripts for TOB infra

* ToB Frontend related updates

* Update settings.gradle.kts

* Deleted redundant file and minor README change

* Addressing comments in the PR

* Added newline at the end of variables.tf file

* Update README.md

* Updates related to Tour of Beam infrastructure

* Update locals.tf

* Output.tf updates

* Update output.tf

* Updates

* Update main.tf

* Updates to cloudfunctions_bucket variable

* service_account_id changes

* Update main.tf

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Bulk update of terraform scripts

* Update README.md

* Update README.md

* Datastore_namespace updates

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update main.tf

* Update README.md

* Update README.md

* Update README.md

* Some minor TF updates

* Update README.md

* Modify batch IT to use count instead of hash (#26327)

* Modify batch IT to use count instead of hash

* remove unused varaiable

* run spotless, update pipeline state checking

* Update timeout to 45 minutes

* revert timeout, add additional counter to try and pinpoint missing records

* add a log to notify ranges used when workers restart

* change counts from metrics to combiners

* add a window to streaming test

* move the passert to the correct place

* Remove extra counter, apply spotless

* add additional metric to KafkaWriter

* Remove debugging metrics

* verify pipeline is not failed

* remove extra newline

* Revert "Modify batch IT to use count instead of hash (#26327)" (#26466)

This reverts commit 9903b2f.

* Bump Java Dataflow container images (#26459)

* keep retrying mass_comment until it has started all jobs (#26457)

* keep retrying mass_comment until it has started all jobs

* fix lookups

* Add driverJars parameter to JdbcIO. (#25824)

This change allows users to use driver jars saved in GCS. With this change, Dataflow templates will be able to migrate to JdbcIO instead of DynamicJdbcIO.

* [Roll Fwd PR] Rename _namespace to _get_display_data_namespace"" (#26470)

* Update parquetio and textio to work with -beam_strict (#26469)

* use wheel sdk location for PostCommit_Py_Examples (#26473)

* Move back the timeout of Python PostCommit to 4h

* Minor fix on Python PostCommit description strings

* Add recent postcommits to jenkins README

* More user-friendly providers.

* Add yaml preprocessing phases.

* Add flexible windowing syntax to yaml.

* Implement flatten in terms of preprocessor phase.

This composes better with windowing.

* Reword SQL note.

* Make linter happy.

* Survive errors in size estimation in MongoDbIO

* Fix jdbc xlang schema type mismatch (#26480)

* Fix jdbc xlang schema type mismatch

* Also fix fetch_size type mismatch

* Add new fields in the end

* [Python] Add saved_weights example to tf notebook (#26472)

* add saved_weights example to tf notebook

* add description

* updated text blocks

* Update examples/notebooks/beam-ml/run_inference_tensorflow.ipynb

Co-authored-by: Rebecca Szper <98840847+rszper@users.noreply.github.com>

* Update examples/notebooks/beam-ml/run_inference_tensorflow.ipynb

Co-authored-by: Rebecca Szper <98840847+rszper@users.noreply.github.com>

---------

Co-authored-by: Rebecca Szper <98840847+rszper@users.noreply.github.com>

* Revert "Revert "Modify batch IT to use count instead of hash (#26327)" (#26466)" (#26467)

This reverts commit 5c67668.

* Bump github.com/tetratelabs/wazero from 1.0.3 to 1.1.0 in /sdks (#26486)

Bumps [github.com/tetratelabs/wazero](https://github.com/tetratelabs/wazero) from 1.0.3 to 1.1.0.
- [Release notes](https://github.com/tetratelabs/wazero/releases)
- [Commits](wazero/wazero@v1.0.3...v1.1.0)

---
updated-dependencies:
- dependency-name: github.com/tetratelabs/wazero
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* tour of beam integration tests (#25925)

* Integration test to load the default example of the default SDK and change the example (#24730) (#24729)

* Fix formatting and README (#24730)

* Support collection v1.17.0 (#24730)

* LoadingIndicator on chaning examples, remove duplicating licenses (#24730)

* Add a missing license header (#24730)

* Integration test for changing SDK and running code (#24779) (#382)

* Integration test for changing SDK and running code (#24779)

* Rename an integration test (#24779)

* Use enum to switch SDK in integration test (#24779)

* Find SDK in a dropdown by key (#24779)

* Add a TODO (#24779)

* Fix exports (#24779)

* Issue24779 integration changing sdk from 24370 (#387)

* Integration test for changing SDK and running code (#24779)

* Rename an integration test (#24779)

* Use enum to switch SDK in integration test (#24779)

* Find SDK in a dropdown by key (#24779)

* Add a TODO (#24779)

* Fix exports (#24779)

* Integration tests miscellaneous UI (#383)

* miscellaneous ui integration tests

* reverted pubspec.lock

* gradle tasks ordered alhpabetically

* integration tests refactoring

* clean code

* integration tests miscellaneous ui fix pr

* rename method

* added layout adaptivity

* A minor cleanup (#24779)

Co-authored-by: Dmitry Repin <mr.malarg@gmail.com>

* integration tests run and editing

* example selector test

* minor fixes

* rat

* fix pr

* minor

* minor

* rat

* integration test finder written

* integration test minor fixes

* minor fixes

* removed comment

* minor fixes

* playground integration tests minor fixes

* integration test pumpAnSettleNoException

* integration test shortcut refactor

* integration test another changing shortcuts running

* upgrade to flutter 3.7.1

* workaround comment

* playground frontend updated major versions

* issues 25329 25331 25336

* 25329 extract connectivity extension to separate file

* Upgrade Flutter to 3.7.3 in integration tests (#24730)

* Fix integration test (#24730)

* fix cors issue and added mouse scroll to tags

* Upgrade Flutter in Dockerfile (#24720)

* sorting moved to model

* sorting moved to model

* sorting moved to model

* bugs fix

* issue 25278

* fix pr

* quites fix in en.yaml

* Fix not loading default example (#25528)

* fix compile error

* Refactor output tabs, test embedded playground (#25136) (#439)

* Refactor output tabs, test embedded playground (#25136)

* Clean up (#25136)

* Change example paths to IDs in integration tests

* issue25640 tob ci

* fix tob ci

* rename ci process

* test add new line to main

* test add new line to main

* commented unit test run

* issue25640 changed server path

* issue25640 tests on welcome page

* deleted config.g.dart

* issue25640 pr fixes

* Update .github/workflows/tour_of_beam_frontend_test.yml

Co-authored-by: alexeyinkin <leha@inkin.ru>

* Update learning/tour-of-beam/frontend/integration_test/welcome_page_test.dart

Co-authored-by: alexeyinkin <leha@inkin.ru>

* Improve tests (#25640)

* issue25640 tour page tests

* pr fix

* removed import

* pr fix

* fix test

* 25640 fixed pubspec.lock

* issue25640 fix readme

* updated readme

* issue25640 fixed after master merge

* issue25483 ToB pipeline options

* removed unnecesary variable

* pr fix

* Update learning/tour-of-beam/frontend/assets/translations/en.yaml

Co-authored-by: alexeyinkin <leha@inkin.ru>

* playground hides when snippet does not exists

* pipeline options extracted to playground components

* issue25483 pipeline options

* added errors handling, fix pr

* refactoring

* Revert "refactoring"

This reverts commit 1540961.

* removed unnecessary constants

* playground controller in tour notifier becomes nullable

* playground controller returned to non nullable in tour notifier

* playground controller actions

* removed unnecessary code

* tob scaffold wrapped with animated builder

* minor fixes

* partially fixed tests

* Upgrade flutter_code_editor to v0.2.19 (#25640)

* Replace output SelectableText with a CodeField instance (#25640)

* Trigger ToB integration tests (#25640)

* Clean up (#25640)

* Enable manual workflow runs for Playground and ToB integration tests (#25640)

---------

Co-authored-by: Alexey Inkin <alexey.inkin@akvelon.com>
Co-authored-by: alexeyinkin <leha@inkin.ru>

* Eliminate nullness errors from MongoDbIO

* Move provisioned options outside of harness.Main (#26476)

Co-authored-by: lostluck <13907733+lostluck@users.noreply.github.com>

* Add SDF adjacent element test

* Consolidate residual processing

* fix race condition on split boundaries

* deflake fixed window combine tests

* Add comments to job functions.

* Make int64check per window.

* Update sdks/go/pkg/beam/runners/prism/internal/worker/bundle.go

Co-authored-by: Ritesh Ghorse <riteshghorse@gmail.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: lostluck <13907733+lostluck@users.noreply.github.com>
Co-authored-by: Danny McCormick <dannymccormick@google.com>
Co-authored-by: Anand Inguva <anandinguva98@gmail.com>
Co-authored-by: Anand Inguva <34158215+AnandInguva@users.noreply.github.com>
Co-authored-by: Darkhan Nausharipov <31556582+nausharipov@users.noreply.github.com>
Co-authored-by: darkhan.nausharipov <darkhan.nausharipov@kzn.akvelon.com>
Co-authored-by: Robert Bradshaw <robertwb@gmail.com>
Co-authored-by: Andrei Gurau <andreigurau@google.com>
Co-authored-by: Ritesh Ghorse <riteshghorse@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: ruslan-ikhsan <114978215+ruslan-ikhsan@users.noreply.github.com>
Co-authored-by: johnjcasey <95318300+johnjcasey@users.noreply.github.com>
Co-authored-by: Yi Hu <yathu@google.com>
Co-authored-by: Pranav Bhandari <bhandari.pranav22@gmail.com>
Co-authored-by: Svetak Sundhar <svetaksundhar@google.com>
Co-authored-by: Jeremy Edwards <jeremyje@gmail.com>
Co-authored-by: Kenneth Knowles <klk@google.com>
Co-authored-by: Rebecca Szper <98840847+rszper@users.noreply.github.com>
Co-authored-by: Dmitry Repin <mr.malarg@gmail.com>
Co-authored-by: Alexey Inkin <alexey.inkin@akvelon.com>
Co-authored-by: alexeyinkin <leha@inkin.ru>
pl04351820 pushed a commit to pl04351820/beam that referenced this pull request Dec 20, 2023
Source-Link: googleapis/synthtool@26558ba
Post-Processor: gcr.io/repo-automation-bots/owlbot-python:latest@sha256:99d90d097e4a4710cc8658ee0b5b963f4426d0e424819787c3ac1405c9a26719

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
cushon pushed a commit to cushon/beam that referenced this pull request May 24, 2024
* Add channel split separations.

* Use channel leases for parallel processing.

* move data wait to chan + atomic

* Progress and split on execute goroutine.

* Convert metrics tests to short ids.

* Add Provision handler w/ capabilities.

* Add artifact validation to run_rc_validation (apache#26407)

* Add artifact validation to run_rc_validation

* config file for artifacts

* use java bom contents for validation

* add pip_pre = True

* Update tox.ini

* Remove out of date experimental warning on SDFs (apache#26450)

* [ToBF] Refinement 26.04.23 (apache#26428)

* content tree rebuilds on sdk change

* expandable parent node widget

* comment fix

* comment fix

---------

Co-authored-by: darkhan.nausharipov <darkhan.nausharipov@kzn.akvelon.com>

* Allow implicit flattening for yaml inputs. (apache#26423)

* Add field annotations for high-priority Syndeo schema transforms (apache#26384)

add changes

* [Go SDK] Timers with new datalayer (apache#26101)

* added timer package

* add timer changes and merged with rebo's pr

* timer fired in stateful

* error setting new timer in ontimer

* looping timers work

* send fv instead of bytes

* changes to coder/pardo

* works for all cases, only cleanup left

* remove comments and validate onTimer

* generic coder for user key

* fixes coder end to end

* remove logs

* add unit test and refactor

* add docs

* new example

* fix static lint

* support emitters

* allow input col of CoGBK as well

* unit tests, periodic impulse, minor refactor

* update PipelineTimer interface, minor refactor, doc comment for example

* add warn message

* single edge timer coder, rm kv coder check, cache encoder,decoder

* Update chromedriver-binary requirement in /sdks/python

Updates the requirements on [chromedriver-binary](https://github.com/danielkaiser/python-chromedriver-binary) to permit the latest version.
- [Release notes](https://github.com/danielkaiser/python-chromedriver-binary/releases)
- [Commits](danielkaiser/python-chromedriver-binary@v100.0.4896.20.0...v113.0.5672.24.0)

---
updated-dependencies:
- dependency-name: chromedriver-binary
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>

* Automation: Tour of Beam infrastructure deployment (apache#25793)

* Added Terraform scripts for TOB infra

* ToB Frontend related updates

* Update settings.gradle.kts

* Deleted redundant file and minor README change

* Addressing comments in the PR

* Added newline at the end of variables.tf file

* Update README.md

* Updates related to Tour of Beam infrastructure

* Update locals.tf

* Output.tf updates

* Update output.tf

* Updates

* Update main.tf

* Updates to cloudfunctions_bucket variable

* service_account_id changes

* Update main.tf

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Bulk update of terraform scripts

* Update README.md

* Update README.md

* Datastore_namespace updates

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update main.tf

* Update README.md

* Update README.md

* Update README.md

* Some minor TF updates

* Update README.md

* Modify batch IT to use count instead of hash (apache#26327)

* Modify batch IT to use count instead of hash

* remove unused varaiable

* run spotless, update pipeline state checking

* Update timeout to 45 minutes

* revert timeout, add additional counter to try and pinpoint missing records

* add a log to notify ranges used when workers restart

* change counts from metrics to combiners

* add a window to streaming test

* move the passert to the correct place

* Remove extra counter, apply spotless

* add additional metric to KafkaWriter

* Remove debugging metrics

* verify pipeline is not failed

* remove extra newline

* Revert "Modify batch IT to use count instead of hash (apache#26327)" (apache#26466)

This reverts commit 9903b2f.

* Bump Java Dataflow container images (apache#26459)

* keep retrying mass_comment until it has started all jobs (apache#26457)

* keep retrying mass_comment until it has started all jobs

* fix lookups

* Add driverJars parameter to JdbcIO. (apache#25824)

This change allows users to use driver jars saved in GCS. With this change, Dataflow templates will be able to migrate to JdbcIO instead of DynamicJdbcIO.

* [Roll Fwd PR] Rename _namespace to _get_display_data_namespace"" (apache#26470)

* Update parquetio and textio to work with -beam_strict (apache#26469)

* use wheel sdk location for PostCommit_Py_Examples (apache#26473)

* Move back the timeout of Python PostCommit to 4h

* Minor fix on Python PostCommit description strings

* Add recent postcommits to jenkins README

* More user-friendly providers.

* Add yaml preprocessing phases.

* Add flexible windowing syntax to yaml.

* Implement flatten in terms of preprocessor phase.

This composes better with windowing.

* Reword SQL note.

* Make linter happy.

* Survive errors in size estimation in MongoDbIO

* Fix jdbc xlang schema type mismatch (apache#26480)

* Fix jdbc xlang schema type mismatch

* Also fix fetch_size type mismatch

* Add new fields in the end

* [Python] Add saved_weights example to tf notebook (apache#26472)

* add saved_weights example to tf notebook

* add description

* updated text blocks

* Update examples/notebooks/beam-ml/run_inference_tensorflow.ipynb

Co-authored-by: Rebecca Szper <98840847+rszper@users.noreply.github.com>

* Update examples/notebooks/beam-ml/run_inference_tensorflow.ipynb

Co-authored-by: Rebecca Szper <98840847+rszper@users.noreply.github.com>

---------

Co-authored-by: Rebecca Szper <98840847+rszper@users.noreply.github.com>

* Revert "Revert "Modify batch IT to use count instead of hash (apache#26327)" (apache#26466)" (apache#26467)

This reverts commit 5c67668.

* Bump github.com/tetratelabs/wazero from 1.0.3 to 1.1.0 in /sdks (apache#26486)

Bumps [github.com/tetratelabs/wazero](https://github.com/tetratelabs/wazero) from 1.0.3 to 1.1.0.
- [Release notes](https://github.com/tetratelabs/wazero/releases)
- [Commits](wazero/wazero@v1.0.3...v1.1.0)

---
updated-dependencies:
- dependency-name: github.com/tetratelabs/wazero
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* tour of beam integration tests (apache#25925)

* Integration test to load the default example of the default SDK and change the example (apache#24730) (apache#24729)

* Fix formatting and README (apache#24730)

* Support collection v1.17.0 (apache#24730)

* LoadingIndicator on chaning examples, remove duplicating licenses (apache#24730)

* Add a missing license header (apache#24730)

* Integration test for changing SDK and running code (apache#24779) (apache#382)

* Integration test for changing SDK and running code (apache#24779)

* Rename an integration test (apache#24779)

* Use enum to switch SDK in integration test (apache#24779)

* Find SDK in a dropdown by key (apache#24779)

* Add a TODO (apache#24779)

* Fix exports (apache#24779)

* Issue24779 integration changing sdk from 24370 (apache#387)

* Integration test for changing SDK and running code (apache#24779)

* Rename an integration test (apache#24779)

* Use enum to switch SDK in integration test (apache#24779)

* Find SDK in a dropdown by key (apache#24779)

* Add a TODO (apache#24779)

* Fix exports (apache#24779)

* Integration tests miscellaneous UI (apache#383)

* miscellaneous ui integration tests

* reverted pubspec.lock

* gradle tasks ordered alhpabetically

* integration tests refactoring

* clean code

* integration tests miscellaneous ui fix pr

* rename method

* added layout adaptivity

* A minor cleanup (apache#24779)

Co-authored-by: Dmitry Repin <mr.malarg@gmail.com>

* integration tests run and editing

* example selector test

* minor fixes

* rat

* fix pr

* minor

* minor

* rat

* integration test finder written

* integration test minor fixes

* minor fixes

* removed comment

* minor fixes

* playground integration tests minor fixes

* integration test pumpAnSettleNoException

* integration test shortcut refactor

* integration test another changing shortcuts running

* upgrade to flutter 3.7.1

* workaround comment

* playground frontend updated major versions

* issues 25329 25331 25336

* 25329 extract connectivity extension to separate file

* Upgrade Flutter to 3.7.3 in integration tests (apache#24730)

* Fix integration test (apache#24730)

* fix cors issue and added mouse scroll to tags

* Upgrade Flutter in Dockerfile (apache#24720)

* sorting moved to model

* sorting moved to model

* sorting moved to model

* bugs fix

* issue 25278

* fix pr

* quites fix in en.yaml

* Fix not loading default example (apache#25528)

* fix compile error

* Refactor output tabs, test embedded playground (apache#25136) (apache#439)

* Refactor output tabs, test embedded playground (apache#25136)

* Clean up (apache#25136)

* Change example paths to IDs in integration tests

* issue25640 tob ci

* fix tob ci

* rename ci process

* test add new line to main

* test add new line to main

* commented unit test run

* issue25640 changed server path

* issue25640 tests on welcome page

* deleted config.g.dart

* issue25640 pr fixes

* Update .github/workflows/tour_of_beam_frontend_test.yml

Co-authored-by: alexeyinkin <leha@inkin.ru>

* Update learning/tour-of-beam/frontend/integration_test/welcome_page_test.dart

Co-authored-by: alexeyinkin <leha@inkin.ru>

* Improve tests (apache#25640)

* issue25640 tour page tests

* pr fix

* removed import

* pr fix

* fix test

* 25640 fixed pubspec.lock

* issue25640 fix readme

* updated readme

* issue25640 fixed after master merge

* issue25483 ToB pipeline options

* removed unnecesary variable

* pr fix

* Update learning/tour-of-beam/frontend/assets/translations/en.yaml

Co-authored-by: alexeyinkin <leha@inkin.ru>

* playground hides when snippet does not exists

* pipeline options extracted to playground components

* issue25483 pipeline options

* added errors handling, fix pr

* refactoring

* Revert "refactoring"

This reverts commit 1540961.

* removed unnecessary constants

* playground controller in tour notifier becomes nullable

* playground controller returned to non nullable in tour notifier

* playground controller actions

* removed unnecessary code

* tob scaffold wrapped with animated builder

* minor fixes

* partially fixed tests

* Upgrade flutter_code_editor to v0.2.19 (apache#25640)

* Replace output SelectableText with a CodeField instance (apache#25640)

* Trigger ToB integration tests (apache#25640)

* Clean up (apache#25640)

* Enable manual workflow runs for Playground and ToB integration tests (apache#25640)

---------

Co-authored-by: Alexey Inkin <alexey.inkin@akvelon.com>
Co-authored-by: alexeyinkin <leha@inkin.ru>

* Eliminate nullness errors from MongoDbIO

* Move provisioned options outside of harness.Main (apache#26476)

Co-authored-by: lostluck <13907733+lostluck@users.noreply.github.com>

* Add SDF adjacent element test

* Consolidate residual processing

* fix race condition on split boundaries

* deflake fixed window combine tests

* Add comments to job functions.

* Make int64check per window.

* Update sdks/go/pkg/beam/runners/prism/internal/worker/bundle.go

Co-authored-by: Ritesh Ghorse <riteshghorse@gmail.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: lostluck <13907733+lostluck@users.noreply.github.com>
Co-authored-by: Danny McCormick <dannymccormick@google.com>
Co-authored-by: Anand Inguva <anandinguva98@gmail.com>
Co-authored-by: Anand Inguva <34158215+AnandInguva@users.noreply.github.com>
Co-authored-by: Darkhan Nausharipov <31556582+nausharipov@users.noreply.github.com>
Co-authored-by: darkhan.nausharipov <darkhan.nausharipov@kzn.akvelon.com>
Co-authored-by: Robert Bradshaw <robertwb@gmail.com>
Co-authored-by: Andrei Gurau <andreigurau@google.com>
Co-authored-by: Ritesh Ghorse <riteshghorse@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: ruslan-ikhsan <114978215+ruslan-ikhsan@users.noreply.github.com>
Co-authored-by: johnjcasey <95318300+johnjcasey@users.noreply.github.com>
Co-authored-by: Yi Hu <yathu@google.com>
Co-authored-by: Pranav Bhandari <bhandari.pranav22@gmail.com>
Co-authored-by: Svetak Sundhar <svetaksundhar@google.com>
Co-authored-by: Jeremy Edwards <jeremyje@gmail.com>
Co-authored-by: Kenneth Knowles <klk@google.com>
Co-authored-by: Rebecca Szper <98840847+rszper@users.noreply.github.com>
Co-authored-by: Dmitry Repin <mr.malarg@gmail.com>
Co-authored-by: Alexey Inkin <alexey.inkin@akvelon.com>
Co-authored-by: alexeyinkin <leha@inkin.ru>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants