[BEAM-121] DisplayData tweaks based on transform usage by swegner · Pull Request #83 · apache/beam

swegner · 2016-03-28T20:51:14Z

[BEAM-121] Publish DisplayData from common PTransforms

Add boolean-valued display data.
Implement equality for DislpayData.Item
Add ability to override namespace for included subcomponents.
Additional Matchers for testing display data
Update DisplayData inner class privacy

* Add boolean-valued display data. * Implement equality for DislpayData.Item * Add ability to override namespace for included subcomponents. * Additional Matchers for testing display data * Update DisplayData inner class privacy

swegner · 2016-03-28T20:51:28Z

R: @kennknowles @bjchambers

swegner · 2016-03-28T20:58:14Z

Once this is in, the code to add display data in SDK transforms is ready to go:

kennknowles · 2016-03-31T03:22:54Z

Sorry this has been sitting - I am bringing myself up to speed. Just letting you know it is not forgotten.

bjchambers · 2016-04-01T17:37:32Z

sdks/java/core/src/main/java/com/google/cloud/dataflow/sdk/transforms/display/DisplayData.java


-    private String getLongValue () {
+    String getLongValue () {
      return this.longValue;


getLongValue() (no space before parens)

bjchambers · 2016-04-01T17:40:52Z

This looks good to me. Let's wait to see if Kenn sees anything since he has fresh eyes (may identify other places for improvement).

One question -- do we have a test that takes all the kinds of display data and writes them to JSON and looks at the details of that encoding (eg., whether a number is written as a number or a string, etc.)? This would be valuable, since changes to the JSON format will have the potential to break runners/UIs that depend on the display data coming across in a specific format.

kennknowles · 2016-04-04T18:54:42Z

...core/src/test/java/com/google/cloud/dataflow/sdk/transforms/display/DisplayDataMatchers.java


 package com.google.cloud.dataflow.sdk.transforms.display;

+import static org.hamcrest.Matchers.allOf;


Today we have no user-facing APIs that link with Hamcrest AFAIK. I think we are trying to sort out a good way to do that without causing dependency hell. (We redacted SerializableMatchers to be package-private because of this issue; we have a lot of value to add by solving it).

It looks like you might be able to get away with submitting it package-private for now.

I see a few other matchers classes exposed:

DataflowMatchers

TestUtils.KVMatcher

WindowMatchers

Should these all be moved internal? I can follow-up with another PR.

I chatted with @lukecwik on this; since DisplayDataMatchers will be packaged in the test jar, it should be ok. This is the case with the others listed above as well.

swegner · 2016-04-04T23:02:42Z

re @bjchambers

do we have a test that takes all the kinds of display data and writes them to JSON and looks at the details of that encoding (eg., whether a number is written as a number or a string, etc.)?

Yes, this exists; check out DataflowPipelineTranslatorTest#testStepDisplayData().

…ydata-tweaks

bjchambers · 2016-04-05T16:24:41Z

LGTM.

bjchambers · 2016-04-05T16:25:45Z

re: testStepDisplayData: It looks like that tests that the constructed Maps are equal, rather than the underlying JSON. Depending on the requirements of consumers, it may be good to actually test that the JSON is exactly the same (since some parsers may be more/less strict about spaces, etc.). Definitely not required in this PR though.

I'll merge this.

Hadoop string sink

…s on Playground Backend * [BEAM-13251][Playground] fix all staticcheck issues * [Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83) * Update workflows for playground * Attempt to fix tests * Remove continue on error to catch errors * Fix linter problem for backend dockerfile * Update folder to run backend go linter * Moved flutter test to execution via gradle tasks * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * [BEAM-13251][Playground] merge with master Co-authored-by: Sergey Kalinin <91209855+snkalinin@users.noreply.github.com> Co-authored-by: Ilya <ilya.kozyrev@akvelon.com>

…of MustParse with Parse for all API methods * [BEAM-13177][Playground] change uuid.MustParse() to uuid.Parse() * [BEAM-13177][Playground] fix typos * [Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83) * Update workflows for playground * Attempt to fix tests * Remove continue on error to catch errors * Fix linter problem for backend dockerfile * Update folder to run backend go linter * Moved flutter test to execution via gradle tasks * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. Co-authored-by: Sergey Kalinin <91209855+snkalinin@users.noreply.github.com> Co-authored-by: Ilya <ilya.kozyrev@akvelon.com>

…eaming output on the frontend * [BEAM-13120] update playground statuses * [BEAM-13210]: add missing statuses * [Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83) * Update workflows for playground * Attempt to fix tests * Remove continue on error to catch errors * Fix linter problem for backend dockerfile * Update folder to run backend go linter * Moved flutter test to execution via gradle tasks * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * [BEAM-13210]: add task to generate config using gradle properties * [BEAM-13210]: add licence to gradle.properties * [BEAM-13210]: update generated config * [BEAM-13210]: update generate config gradle task * [BEAM-13210]: rewrite createconfig to groovy Co-authored-by: Sergey Kalinin <91209855+snkalinin@users.noreply.github.com> Co-authored-by: Ilya <ilya.kozyrev@akvelon.com> Co-authored-by: Aydar Zainutdinov <aydar.zaynutdinov@akvelon.com>

…xamples api * Add reading of precompiled objects from public bucket * Change the enum PrecompiledObjectType * RAT for pb.dart * [Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83) * Update workflows for playground * Attempt to fix tests * Remove continue on error to catch errors * Fix linter problem for backend dockerfile * Update folder to run backend go linter * Moved flutter test to execution via gradle tasks * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * Add info about meta file Co-authored-by: Sergey Kalinin <91209855+snkalinin@users.noreply.github.com> Co-authored-by: Ilya <ilya.kozyrev@akvelon.com> Co-authored-by: Aydar Zainutdinov <aydar.zaynutdinov@akvelon.com>

…o send found tagged file to the backend * [Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83) * Update workflows for playground * Attempt to fix tests * Remove continue on error to catch errors * Fix linter problem for backend dockerfile * Update folder to run backend go linter * Moved flutter test to execution via gradle tasks * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * Add client to send found tagged file to the backend; Add python proto files; Co-authored-by: Sergey Kalinin <91209855+snkalinin@users.noreply.github.com> Co-authored-by: Ilya <ilya.kozyrev@akvelon.com> Co-authored-by: Aydar Zainutdinov <aydar.zaynutdinov@akvelon.com> Co-authored-by: daria.malkova <daria.malkova@akvelon.com>

…on and metadata * [Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83) * Update workflows for playground * Attempt to fix tests * Remove continue on error to catch errors * Fix linter problem for backend dockerfile * Update folder to run backend go linter * Moved flutter test to execution via gradle tasks * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * [BEAM-13281]: playground - update favicon and metadata Co-authored-by: Sergey Kalinin <91209855+snkalinin@users.noreply.github.com> Co-authored-by: Ilya <ilya.kozyrev@akvelon.com> Co-authored-by: Aydar Zainutdinov <aydar.zaynutdinov@akvelon.com>

…arch & filter logic * [BEAM-12967]: Created Examples Component * [BEAM-12967]: Wrapped CategoryBubble and ExpansionPanelItem in MouseRegion * [BEAM-13064]: Implemented Search field and Filter by type components logic * [BEAM-13064]: Fixed names to clarify code, refactored Category model (added const keyword) * [BEAM-13064]: Increased test coverage of Example Dropdown module * [BEAM-12967] Added generated files * [BEAM-12967]: Added license to generated files * [BEAM-13064]: Fixed sorting functionality, updated tests & mocks * [BEAM-13064]: Refactored code * [BEAM-12967]: Deleted duplicate files * [BEAM-13064] Fixes after merge * [Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83) * Update workflows for playground * Attempt to fix tests * Remove continue on error to catch errors * Fix linter problem for backend dockerfile * Update folder to run backend go linter * Moved flutter test to execution via gradle tasks * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * [BEAM-13064] Fixed analyze issues * [BEAM-13064] Fixed example selector state test Co-authored-by: Sergey Kalinin <91209855+snkalinin@users.noreply.github.com> Co-authored-by: Ilya <ilya.kozyrev@akvelon.com> Co-authored-by: Aydar Zainutdinov <aydar.zaynutdinov@akvelon.com> Co-authored-by: daria.malkova <daria.malkova@akvelon.com>

…toring LifeCycle * [Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83) * Update workflows for playground * Attempt to fix tests * Remove continue on error to catch errors * Fix linter problem for backend dockerfile * Update folder to run backend go linter * Moved flutter test to execution via gradle tasks * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * Rename LF executable -> source * Reanme LF compile -> executables Co-authored-by: Sergey Kalinin <91209855+snkalinin@users.noreply.github.com> Co-authored-by: Ilya <ilya.kozyrev@akvelon.com> Co-authored-by: Aydar Zainutdinov <aydar.zaynutdinov@akvelon.com> Co-authored-by: Pavel Avilov <pavel.avilov@akvelon.com>

…e redundant code from controller * [Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83) * Update workflows for playground * Attempt to fix tests * Remove continue on error to catch errors * Fix linter problem for backend dockerfile * Update folder to run backend go linter * Moved flutter test to execution via gradle tasks * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * Remove redundant methods from controller, move precompiled objects method to separate package * fix RAT Co-authored-by: Sergey Kalinin <91209855+snkalinin@users.noreply.github.com> Co-authored-by: Ilya <ilya.kozyrev@akvelon.com> Co-authored-by: Aydar Zainutdinov <aydar.zaynutdinov@akvelon.com> Co-authored-by: Pavel Avilov <pavel.avilov@akvelon.com>

…ructure to find and validate tagged files * [Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83) * Update workflows for playground * Attempt to fix tests * Remove continue on error to catch errors * Fix linter problem for backend dockerfile * Update folder to run backend go linter * Moved flutter test to execution via gradle tasks * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * [BEAM-13129][Playground] Added base structure to find and verify beam examples Co-authored-by: Sergey Kalinin <91209855+snkalinin@users.noreply.github.com> Co-authored-by: Ilya <ilya.kozyrev@akvelon.com> Co-authored-by: daria.malkova <daria.malkova@akvelon.com> Co-authored-by: Pavel Avilov <pavel.avilov@akvelon.com>

…eaming output for the run code logs * [Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83) * Update workflows for playground * Attempt to fix tests * Remove continue on error to catch errors * Fix linter problem for backend dockerfile * Update folder to run backend go linter * Moved flutter test to execution via gradle tasks * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * [BEAM-13156][Playground] implementation of streaming output for run code output Co-authored-by: Sergey Kalinin <91209855+snkalinin@users.noreply.github.com> Co-authored-by: Ilya <ilya.kozyrev@akvelon.com>

…s using the api * [BEAM-12967]: Created Examples Component * [BEAM-12967]: Wrapped CategoryBubble and ExpansionPanelItem in MouseRegion * [BEAM-13064]: Implemented Search field and Filter by type components logic * [BEAM-13064]: Fixed names to clarify code, refactored Category model (added const keyword) * [BEAM-13064]: Increased test coverage of Example Dropdown module * [BEAM-12967] Added generated files * [BEAM-12967]: Added license to generated files * [BEAM-13064]: Fixed sorting functionality, updated tests & mocks * [BEAM-13064]: Refactored code * [BEAM-12967]: Deleted duplicate files * [BEAM-13101]: Implementing new Example structure * [BEAM-13101] Added getPrecompiledOutputs method * [BEAM-13101]: Added default examples request, changed RunCode function to use precompiled Outputs * [BEAM-13101]: Fixed use of precompiledOutputs * [BEAM-13101]: Auto-formatted files, fixed unit tests * Change protobuf * [BEAM-13101] generate proto * [BEAM-13101] revert generate proto * [BEAM-13101] Reworked examples structure * [BEAM-13101] Changed example structure, regenerate proto * [BEAM-13101] Fixes after merge request * [BEAM-13101] Example structure fixes, added licenses * [BEAM-13101] Removed unnecessary file and comment * [Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83) * Update workflows for playground * Attempt to fix tests * Remove continue on error to catch errors * Fix linter problem for backend dockerfile * Update folder to run backend go linter * Moved flutter test to execution via gradle tasks * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * [BEAM-13101]: fix tests and refactor examples api * [BEAM-13101]: fix linter issues * [BEAM-13101]: add licence * [BEAM-13101]: format code * [BEAM-13101]: add missing mock file * [BEAM-13101] Updated .proto file, regenerated proto * [BEAM-13101] Fixed import in grpc example client Co-authored-by: Alexander Zhuravlev <stimbai@mail.ru> Co-authored-by: daria-malkova <daria.malkova@akvelon.com> Co-authored-by: Alexander <alexander.zhuravlev@akvelon.com> Co-authored-by: Sergey Kalinin <91209855+snkalinin@users.noreply.github.com> Co-authored-by: Ilya <ilya.kozyrev@akvelon.com> Co-authored-by: Aydar Zainutdinov <aydar.zaynutdinov@akvelon.com> Co-authored-by: Pavel Avilov <pavel.avilov@akvelon.com>

…oundPrecommit to run on GitHub Actions for pull requests * [Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83) * Update workflows for playground * Attempt to fix tests * Remove continue on error to catch errors * Fix linter problem for backend dockerfile * Update folder to run backend go linter * Moved flutter test to execution via gradle tasks * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * Create precommit tasks for backend and frontend, replaced separate tests in actions to running precommit * Added linter to preCommit, updated install terraform via github actions * Fix typos and install of terraform Co-authored-by: Ilya <ilya.kozyrev@akvelon.com> Co-authored-by: Aydar Zainutdinov <aydar.zaynutdinov@akvelon.com>

…lement copying a prepared go.mod * [Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83) * Update workflows for playground * Attempt to fix tests * Remove continue on error to catch errors * Fix linter problem for backend dockerfile * Update folder to run backend go linter * Moved flutter test to execution via gradle tasks * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * Implement copying a prepared go.mod and go.sum Co-authored-by: Sergey Kalinin <91209855+snkalinin@users.noreply.github.com> Co-authored-by: Ilya <ilya.kozyrev@akvelon.com> Co-authored-by: Aydar Zainutdinov <aydar.zaynutdinov@akvelon.com> Co-authored-by: daria.malkova <daria.malkova@akvelon.com>

… example verification method * [Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83) * Update workflows for playground * Attempt to fix tests * Remove continue on error to catch errors * Fix linter problem for backend dockerfile * Update folder to run backend go linter * Moved flutter test to execution via gradle tasks * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * Implement an example verification method; Setup logger; Co-authored-by: Sergey Kalinin <91209855+snkalinin@users.noreply.github.com> Co-authored-by: Ilya <ilya.kozyrev@akvelon.com> Co-authored-by: Aydar Zainutdinov <aydar.zaynutdinov@akvelon.com> Co-authored-by: daria.malkova <daria.malkova@akvelon.com>

…rr from stdout for code processing steps * [Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83) * Update workflows for playground * Attempt to fix tests * Remove continue on error to catch errors * Fix linter problem for backend dockerfile * Update folder to run backend go linter * Moved flutter test to execution via gradle tasks * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * [BEAM-13208][Playground] separated stdErr and stdOut for compile step Co-authored-by: Sergey Kalinin <91209855+snkalinin@users.noreply.github.com> Co-authored-by: Ilya <ilya.kozyrev@akvelon.com> Co-authored-by: daria.malkova <daria.malkova@akvelon.com> Co-authored-by: Pavel Avilov <pavel.avilov@akvelon.com>

…CI checks * [Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83) * Update workflows for playground * Attempt to fix tests * Remove continue on error to catch errors * Fix linter problem for backend dockerfile * Update folder to run backend go linter * Moved flutter test to execution via gradle tasks * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * [BEAM-13323][Playground] added processing of err case for utils.SetToCache() method in GetRunOutput() api method Co-authored-by: Sergey Kalinin <91209855+snkalinin@users.noreply.github.com> Co-authored-by: Ilya <ilya.kozyrev@akvelon.com> Co-authored-by: daria.malkova <daria.malkova@akvelon.com> Co-authored-by: Pavel Avilov <pavel.avilov@akvelon.com> Co-authored-by: Aydar Farrakhov <stranniknm@gmail.com>

…n SDK * [Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83) * Update workflows for playground * Attempt to fix tests * Remove continue on error to catch errors * Fix linter problem for backend dockerfile * Update folder to run backend go linter * Moved flutter test to execution via gradle tasks * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * draft * Rename LF executable -> source * Reanme LF compile -> executables * working python sdk * Update playground/backend/internal/fs_tool/python_fs_test.go Co-authored-by: Aydar Zainutdinov <aydar.zaynutdinov@akvelon.com> * Update playground/backend/internal/fs_tool/python_fs_test.go Co-authored-by: Aydar Zainutdinov <aydar.zaynutdinov@akvelon.com> * refactoring + adding dockerfile * add preparators array (will be filled later) * fix indentation * [BEAM-13307][Playground] Fix tests; Update processing of the compile step for python sdk; Co-authored-by: Sergey Kalinin <91209855+snkalinin@users.noreply.github.com> Co-authored-by: Ilya <ilya.kozyrev@akvelon.com> Co-authored-by: Aydar Zainutdinov <aydar.zaynutdinov@akvelon.com> Co-authored-by: Pavel Avilov <pavel.avilov@akvelon.com> Co-authored-by: Aydar Farrakhov <stranniknm@gmail.com>

…Preparators * [Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83) * Update workflows for playground * Attempt to fix tests * Remove continue on error to catch errors * Fix linter problem for backend dockerfile * Update folder to run backend go linter * Moved flutter test to execution via gradle tasks * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * [BEAM-13306][Playground] Add using of preparators for code processing * [BEAM-13306][Playground] Add python preparators Co-authored-by: Sergey Kalinin <91209855+snkalinin@users.noreply.github.com> Co-authored-by: Ilya <ilya.kozyrev@akvelon.com> Co-authored-by: daria.malkova <daria.malkova@akvelon.com> Co-authored-by: Pavel Avilov <pavel.avilov@akvelon.com> Co-authored-by: Aydar Farrakhov <stranniknm@gmail.com>

…to find all tagged files * [Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83) * Update workflows for playground * Attempt to fix tests * Remove continue on error to catch errors * Fix linter problem for backend dockerfile * Update folder to run backend go linter * Moved flutter test to execution via gradle tasks * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * [BEAM-13135][Playground] implemented method to find all tagged files; added categories.yaml file; added requirements.txt file; * [BEAM-13135][Playground] updated categories Co-authored-by: Sergey Kalinin <91209855+snkalinin@users.noreply.github.com> Co-authored-by: Ilya <ilya.kozyrev@akvelon.com> Co-authored-by: daria.malkova <daria.malkova@akvelon.com> Co-authored-by: Pavel Avilov <pavel.avilov@akvelon.com> Co-authored-by: Aydar Farrakhov <stranniknm@gmail.com>

…end doesn't send correct status when use Redis cache instead of local cache * [Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83) * Update workflows for playground * Attempt to fix tests * Remove continue on error to catch errors * Fix linter problem for backend dockerfile * Update folder to run backend go linter * Moved flutter test to execution via gradle tasks * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * [BEAM-13347][Playground] Fix to use redis cache Co-authored-by: Sergey Kalinin <91209855+snkalinin@users.noreply.github.com> Co-authored-by: Ilya <ilya.kozyrev@akvelon.com> Co-authored-by: daria.malkova <daria.malkova@akvelon.com> Co-authored-by: Pavel Avilov <pavel.avilov@akvelon.com> Co-authored-by: Aydar Farrakhov <stranniknm@gmail.com>

…_statuses method * [Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83) * Update workflows for playground * Attempt to fix tests * Remove continue on error to catch errors * Fix linter problem for backend dockerfile * Update folder to run backend go linter * Moved flutter test to execution via gradle tasks * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * [BEAM-13267][Playground] implemented a method to send examples to the backend to verify them Co-authored-by: Sergey Kalinin <91209855+snkalinin@users.noreply.github.com> Co-authored-by: Ilya <ilya.kozyrev@akvelon.com> Co-authored-by: daria.malkova <daria.malkova@akvelon.com> Co-authored-by: Pavel Avilov <pavel.avilov@akvelon.com> Co-authored-by: Aydar Farrakhov <stranniknm@gmail.com>

…utor builder * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * Refactor usage of executors * Merge conflict Co-authored-by: Ilya <ilya.kozyrev@akvelon.com>

…ting * [Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83) * Update workflows for playground * Attempt to fix tests * Remove continue on error to catch errors * Fix linter problem for backend dockerfile * Update folder to run backend go linter * Moved flutter test to execution via gradle tasks * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * [BEAM-13241] Added routing by sdk * [BEAM-13241] Moved urls to config.g.dart, changed tests * [BEAM-13241] Changed grpc_example_client route to backend JAVA route * [BEAM-13241] Fixed routing, added workaround channel * [BEAM-13241] fix linter issues * [BEAM-13241] add licences to mocks Co-authored-by: Sergey Kalinin <91209855+snkalinin@users.noreply.github.com> Co-authored-by: Ilya <ilya.kozyrev@akvelon.com> Co-authored-by: Aydar Zainutdinov <aydar.zaynutdinov@akvelon.com> Co-authored-by: daria.malkova <daria.malkova@akvelon.com> Co-authored-by: Pavel Avilov <pavel.avilov@akvelon.com> Co-authored-by: Aydar Farrakhov <stranniknm@gmail.com>

… precompiled examples * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * implement cd step * [BEAM-13135][Playground] implemented method to find all tagged files; added categories.yaml file; added requirements.txt file; * [BEAM-13267][Playground] implemented a method to send examples to the backend to verify them * CD step * Fix merge conflict Co-authored-by: Ilya <ilya.kozyrev@akvelon.com> Co-authored-by: AydarZaynutdinov <aydar.zaynutdinov@akvelon.com>

…e CI/CD To Remove Github Actions That Are Not Permitted To Use In Beam Repo * [Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83) * Update workflows for playground * Attempt to fix tests * Remove continue on error to catch errors * Fix linter problem for backend dockerfile * Update folder to run backend go linter * Moved flutter test to execution via gradle tasks * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * Replace actions that are not permitted to be used in Apache repo Co-authored-by: Ilya <ilya.kozyrev@akvelon.com> Co-authored-by: Aydar Zainutdinov <aydar.zaynutdinov@akvelon.com> Co-authored-by: daria.malkova <daria.malkova@akvelon.com> Co-authored-by: Pavel Avilov <pavel.avilov@akvelon.com> Co-authored-by: Aydar Farrakhov <stranniknm@gmail.com>

@test

…Unit-tests * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * Add java unit tests support * Added @test for java unit-tests * Fix rebase conflicts * Fix concurrent write to map * Fix unit-test with pipeline options Co-authored-by: Ilya <ilya.kozyrev@akvelon.com>

…s With Installing Flutter On GitHub Actions * [Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83) * Update workflows for playground * Attempt to fix tests * Remove continue on error to catch errors * Fix linter problem for backend dockerfile * Update folder to run backend go linter * Moved flutter test to execution via gradle tasks * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * Fix locations of flutter executables Co-authored-by: Ilya <ilya.kozyrev@akvelon.com> Co-authored-by: Aydar Zainutdinov <aydar.zaynutdinov@akvelon.com> Co-authored-by: daria.malkova <daria.malkova@akvelon.com> Co-authored-by: Pavel Avilov <pavel.avilov@akvelon.com> Co-authored-by: Aydar Farrakhov <stranniknm@gmail.com>

…it tests * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * Add go unit tests support * unit tests fixes * Update playground/backend/internal/executors/executor_test.go Co-authored-by: Aydar Zainutdinov <aydar.zaynutdinov@akvelon.com> * Added verbose flag for go unit tests Co-authored-by: Ilya <ilya.kozyrev@akvelon.com> Co-authored-by: Aydar Zainutdinov <aydar.zaynutdinov@akvelon.com> Co-authored-by: Artur Khanin <artur.khanin@akvelon.com>

…n unit tests * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * Add go unit tests support * Add go unit tests support * Merge conflicts * support Python unit tests * bugfix * Add go unit tests support * unit tests fixes * Update playground/backend/internal/executors/executor_test.go Co-authored-by: Aydar Zainutdinov <aydar.zaynutdinov@akvelon.com> * code processing small refactoring * Add processing of an error Co-authored-by: Ilya <ilya.kozyrev@akvelon.com> Co-authored-by: Aydar Zainutdinov <aydar.zaynutdinov@akvelon.com>

…configuration to enable embedding iframes * [Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83) * Update workflows for playground * Attempt to fix tests * Remove continue on error to catch errors * Fix linter problem for backend dockerfile * Update folder to run backend go linter * Moved flutter test to execution via gradle tasks * Revert "[Playground][BEAM-12941][Bugfix] Fix workflows for playground applications (#83)" (#88) This reverts commit b73f5f7. * Added support for embedded iframe and also gzip compression of assets * Add gzip_static config to nginx Co-authored-by: Ilya <ilya.kozyrev@akvelon.com> Co-authored-by: Aydar Zainutdinov <aydar.zaynutdinov@akvelon.com> Co-authored-by: daria.malkova <daria.malkova@akvelon.com> Co-authored-by: Pavel Avilov <pavel.avilov@akvelon.com> Co-authored-by: Aydar Farrakhov <stranniknm@gmail.com>

DisplayData tweaks based on transform usage.

a52c1b6

* Add boolean-valued display data. * Implement equality for DislpayData.Item * Add ability to override namespace for included subcomponents. * Additional Matchers for testing display data * Update DisplayData inner class privacy

swegner changed the title ~~DisplayData tweaks based on transform usage~~ [BEAM-121] DisplayData tweaks based on transform usage Mar 28, 2016

bjchambers reviewed Apr 1, 2016
View reviewed changes

kennknowles reviewed Apr 4, 2016
View reviewed changes

swegner added 2 commits April 4, 2016 16:31

fixup: review feedback

ce58008

Merge branch 'master' of github.com:apache/incubator-beam into displa…

b13015c

…ydata-tweaks

asfgit closed this in 2f8ba65 Apr 5, 2016

swegner deleted the displaydata-tweaks branch April 5, 2016 22:28

iemejia referenced this pull request in iemejia/beam Jan 12, 2018

This closes #83

160a5d3

mareksimunek pushed a commit to mareksimunek/beam that referenced this pull request May 9, 2018

Merge pull request apache#83 from seznam/pete/hadoop-string-sink

02cd5d2

Hadoop string sink

hengfengli pushed a commit to hengfengli/beam that referenced this pull request Mar 21, 2022

tests: fix npe in query change stream action test (apache#83)

4e50773

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[BEAM-121] DisplayData tweaks based on transform usage#83

[BEAM-121] DisplayData tweaks based on transform usage#83
swegner wants to merge 3 commits intoapache:masterfrom
swegner:displaydata-tweaks

swegner commented Mar 28, 2016

Uh oh!

swegner commented Mar 28, 2016

Uh oh!

swegner commented Mar 28, 2016

Uh oh!

kennknowles commented Mar 31, 2016

Uh oh!

bjchambers Apr 1, 2016

Uh oh!

bjchambers commented Apr 1, 2016

Uh oh!

kennknowles Apr 4, 2016

Uh oh!

swegner Apr 4, 2016

Uh oh!

swegner Apr 4, 2016

Uh oh!

swegner commented Apr 4, 2016

Uh oh!

bjchambers commented Apr 5, 2016

Uh oh!

bjchambers commented Apr 5, 2016

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants


		package com.google.cloud.dataflow.sdk.transforms.display;

		import static org.hamcrest.Matchers.allOf;

Conversation

swegner commented Mar 28, 2016

Uh oh!

swegner commented Mar 28, 2016

Uh oh!

swegner commented Mar 28, 2016

Uh oh!

kennknowles commented Mar 31, 2016

Uh oh!

bjchambers Apr 1, 2016

Choose a reason for hiding this comment

Uh oh!

bjchambers commented Apr 1, 2016

Uh oh!

kennknowles Apr 4, 2016

Choose a reason for hiding this comment

Uh oh!

swegner Apr 4, 2016

Choose a reason for hiding this comment

Uh oh!

swegner Apr 4, 2016

Choose a reason for hiding this comment

Uh oh!

swegner commented Apr 4, 2016

Uh oh!

bjchambers commented Apr 5, 2016

Uh oh!

bjchambers commented Apr 5, 2016

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants