[jira] [Work logged] (BEAM-8354) Add RowJsonSerializer
[ https://issues.apache.org/jira/browse/BEAM-8354?focusedWorklogId=325380=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-325380 ] ASF GitHub Bot logged work on BEAM-8354: Author: ASF GitHub Bot Created on: 08/Oct/19 23:13 Start Date: 08/Oct/19 23:13 Worklog Time Spent: 10m Work Description: kennknowles commented on issue #9732: [BEAM-8354] Add `RowJsonSerializer` URL: https://github.com/apache/beam/pull/9732#issuecomment-539742866 Ah, everything in the `util` package is not for users. (they may or may not be on board with this design decision) This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 325380) Time Spent: 40m (was: 0.5h) > Add RowJsonSerializer > - > > Key: BEAM-8354 > URL: https://issues.apache.org/jira/browse/BEAM-8354 > Project: Beam > Issue Type: New Feature > Components: sdk-java-core >Reporter: Brian Hulette >Assignee: Brian Hulette >Priority: Major > Time Spent: 40m > Remaining Estimate: 0h > > We should add a RowJsonSerializer that is compatible with RowJsonDeserializer -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (BEAM-8354) Add RowJsonSerializer
[ https://issues.apache.org/jira/browse/BEAM-8354?focusedWorklogId=324796=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-324796 ] ASF GitHub Bot logged work on BEAM-8354: Author: ASF GitHub Bot Created on: 08/Oct/19 01:08 Start Date: 08/Oct/19 01:08 Worklog Time Spent: 10m Work Description: TheNeuralBit commented on issue #9732: [BEAM-8354] Add `RowJsonSerializer` URL: https://github.com/apache/beam/pull/9732#issuecomment-539269206 I like the idea of combining serializer and seserializer into an outer class. Would there be any concern with changing the API since `RowJsonDeserializer` has been public for a while? Or is it generally considered ok to change around classes in util? This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 324796) Time Spent: 0.5h (was: 20m) > Add RowJsonSerializer > - > > Key: BEAM-8354 > URL: https://issues.apache.org/jira/browse/BEAM-8354 > Project: Beam > Issue Type: New Feature > Components: sdk-java-core >Reporter: Brian Hulette >Assignee: Brian Hulette >Priority: Major > Time Spent: 0.5h > Remaining Estimate: 0h > > We should add a RowJsonSerializer that is compatible with RowJsonDeserializer -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (BEAM-8354) Add RowJsonSerializer
[ https://issues.apache.org/jira/browse/BEAM-8354?focusedWorklogId=323731=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-323731 ] ASF GitHub Bot logged work on BEAM-8354: Author: ASF GitHub Bot Created on: 04/Oct/19 21:00 Start Date: 04/Oct/19 21:00 Worklog Time Spent: 10m Work Description: TheNeuralBit commented on pull request #9732: [BEAM-8354] Add `RowJsonSerializer` URL: https://github.com/apache/beam/pull/9732 Adds a `RowJsonSerializer`, and renames `RowJsonDeserializerTest` to `RowJsonTest`, as it now includes roundtrip tests from row to json and back. R: @apilloud Post-Commit Tests Status (on master branch) Lang | SDK | Apex | Dataflow | Flink | Gearpump | Samza | Spark --- | --- | --- | --- | --- | --- | --- | --- Go | [](https://builds.apache.org/job/beam_PostCommit_Go/lastCompletedBuild/) | --- | --- | [](https://builds.apache.org/job/beam_PostCommit_Go_VR_Flink/lastCompletedBuild/) | --- | --- | [](https://builds.apache.org/job/beam_PostCommit_Go_VR_Spark/lastCompletedBuild/) Java | [](https://builds.apache.org/job/beam_PostCommit_Java/lastCompletedBuild/) | [](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Apex/lastCompletedBuild/) | [](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Dataflow/lastCompletedBuild/) | [](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Flink/lastCompletedBuild/)[](https://builds.apache.org/job/beam_PostCommit_Java_PVR_Flink_Batch/lastCompletedBuild/)[](https://builds.apache.org/job/beam_PostCommit_Java_PVR_Flink_Streaming/lastCompletedBuild/) | [](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Gearpump/lastCompletedBuild/) | [](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Samza/lastCompletedBuild/) | [](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Spark/lastCompletedBuild/)[](https://builds.apache.org/job/beam_PostCommit_Java_PVR_Spark_Batch/lastCompletedBuild/) Python | [](https://builds.apache.org/job/beam_PostCommit_Python2/lastCompletedBuild/)[](https://builds.apache.org/job/beam_PostCommit_Python35/lastCompletedBuild/)[](https://builds.apache.org/job/beam_PostCommit_Python36/lastCompletedBuild/)[](https://builds.apache.org/job/beam_PostCommit_Python37/lastCompletedBuild/) | --- | [](https://builds.apache.org/job/beam_PostCommit_Py_VR_Dataflow/lastCompletedBuild/)[](https://builds.apache.org/job/beam_PostCommit_Py_ValCont/lastCompletedBuild/) | [