[BEAM-12164] Added integration test for transaction boundaries and transaction ID ordering. Made small fixes in ordered by key integration test. #16977
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ordering. Made small fixes in ordered by key integration test.
|
Run Java PostCommit |
1 similar comment
|
Run Java PostCommit |
|
Seems like Java PostCommit did not run. |
|
Run Java PostCommit |
…ependencies to 24.3.0 * Update GCP Libraries BOM version to 24.3.0 * Update associated dependencies
…ent method to get a default example for each SDKs * Implement method to get a default example for each SDKs * Add error handling * Added saving of precompiled objects catalog to cache at the server startup * Added caching of the catalog only in case of unspecified SDK * Update regarding comments * Update regarding comments * Simplified logging regarding comment * Get defaultExamplePath from the corresponding config * Refactoring code * Add the `link` field to response * Remove gjson; Resolve conflicts; * Refactoring code * Getting default precompiled object from cache * Refactoring code * Added saving of precompiled objects catalog to cache at the server startup * Added caching of the catalog only in case of unspecified SDK * Update regarding comments * Update regarding comments * Simplified logging regarding comment * Updates regarding comments * Update for environment_service_test.go * Get default example from catalog * GetCatalogFromCacheOrStorage method * Update licenses * Update licenses; Resolve conflicts; * [BEAM-13633][Playground] Change saving default precompiled objects to the cache * [BEAM-13633][Playground] Change logic of saving and receiving info about default precompiled objects * [BEAM-13633][Playground] Separate for each sdk * [BEAM-13633][Playground] regenerate proto files * Add code of the default example to response * Revert "Add code of the default example to response" This reverts commit da6baa0. * Refactoring code * Refactoring code; Add test; * Edit commentaries * Refactoring code * Add bucket name to methods Co-authored-by: Artur Khanin <artur.khanin@akvelon.com> Co-authored-by: AydarZaynutdinov <aydar.zaynutdinov@akvelon.com> Co-authored-by: Pavel Avilov <pavel.avilov>
…row and proto (apache#16875) * Adding schema support. * Addressing feedback.
… when FlinkRunner is used (apache#16904)
* [BEAM-13964] Bump kotlin to 1.6.x * [BEAM-13964] Bump kotlin to 1.6.x * [BEAM-13964] fix warnings in Kotlin compilation
…rnings (no-name-in-module/no-value-for-parameter) * [BEAM-13051] Pylint no-name-in-module and no-value-for-parameter warnings enabled * [BEAM-13051] Fixed no-value-for-parameter warning for missing default values * [BEAM-13051] Fixed parameters warnings
Codecov Report
@@ Coverage Diff @@
## master #16977 +/- ##
==========================================
- Coverage 73.94% 73.94% -0.01%
==========================================
Files 669 667 -2
Lines 88158 88071 -87
==========================================
- Hits 65191 65125 -66
+ Misses 21855 21835 -20
+ Partials 1112 1111 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
github-actions
bot
removed
sql
model
examples
elasticsearch
kotlin
python
extensions
go
build
runners
aws
direct
website
docker
labels
|
Run Java PostCommit |
|
LGTM |
|
Run Java PreCommit |
|
LGTM thanks @nancyxu123 ! |
|
Oops - we have test suites that pull this in but it needs to be excluded: https://ci-beam.apache.org/job/beam_PostCommit_Java_DataflowV1/1390/ Filed https://issues.apache.org/jira/browse/BEAM-14151 and gave to Pablo since I couldn't find your account. |
|
Sent out #17156
…On Tue, Mar 22, 2022 at 1:56 PM Kenn Knowles ***@***.***> wrote:
Oops - we have test suites that pull this in but it needs to be excluded:
https://ci-beam.apache.org/job/beam_PostCommit_Java_DataflowV1/1390/
Filed https://issues.apache.org/jira/browse/BEAM-14151 and gave to Pablo
since I couldn't find your account.
—
Reply to this email directly, view it on GitHub
<#16977 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAJ5Z3BDTZD3RTFEYJ6OU43VBIXXPANCNFSM5PUXASVA>
.
You are receiving this because you modified the open/close state.Message
ID: ***@***.***>
|
|
Pablo, it looks like the Java PostCommit fails for some of these ITs with the direct runner, filed https://issues.apache.org/jira/browse/BEAM-14193. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added integration test for transaction boundaries and transaction ID ordering. Made small fixes in ordered by key integration test.