[jira] (BEAM-1354) TextIO should comply with PTransform style guide

2017-01-31 Thread Aviem Zur (JIRA) 
Title: Message Title
 
 
 
 
 
 
 
 
 
 
  
 
 Aviem Zur updated an issue 
 
 
 
 
 
 
 
 
 
 

 
 
 
 
 
 
 
 Beam /  BEAM-1354 
 
 
 
  TextIO should comply with PTransform style guide  
 
 
 
 
 
 
 
 
 

Change By:
 
 Aviem Zur 
 
 
 
 
 
 
 
 
 
 * TextIO.Read,Write.Bound,Unbound - Bound,Unbound are banned names. Instead, TextIO should make the user specify the type parameters explicitly, and have simply TextIO.Read and TextIO.Write themselves be the transform tyles.* Both should take simply String, and not use Coder as a general-purpose serialization mechanism.* * The Javadoc should be changed to reflect this.*  Should perhaps use AutoValue for parameter builders 
 
 
 
 
 
 
 
 
 
 
 
 

 
 Add Comment 
 
 
 
 
 
 
 
 
 
 

 
 
 
 
 
 
 
 
 
 

 This message was sent by Atlassian JIRA (v6.3.15#6346-sha1:dbc023d)

[jira] (BEAM-1354) TextIO should comply with PTransform style guide

2017-01-30 Thread Eugene Kirpichov (JIRA) 
Title: Message Title
 
 
 
 
 
 
 
 
 
 
  
 
 Eugene Kirpichov updated an issue 
 
 
 
 
 
 
 
 
 
 

 
 
 
 
 
 
 
 Beam /  BEAM-1354 
 
 
 
  TextIO should comply with PTransform style guide  
 
 
 
 
 
 
 
 
 

Change By:
 
 Eugene Kirpichov 
 
 
 
 
 
 
 
 
 
 * TextIO. { Read,Write } . { Bound,Unbound }  - Bound,Unbound  are banned names. Instead, TextIO should make the user specify the type parameters explicitly, and have simply TextIO.Read and TextIO.Write themselves be the transform tyles.* Both should take simply String, and not use Coder as a general-purpose serialization mechanism.* Should perhaps use AutoValue for parameter builders 
 
 
 
 
 
 
 
 
 
 
 
 

 
 Add Comment 
 
 
 
 
 
 
 
 
 
 

 
 
 
 
 
 
 
 
 
 

 This message was sent by Atlassian JIRA (v6.3.15#6346-sha1:dbc023d)

[jira] (BEAM-1354) TextIO should comply with PTransform style guide

2017-01-30 Thread Eugene Kirpichov (JIRA) 
Title: Message Title
 
 
 
 
 
 
 
 
 
 
  
 
 Eugene Kirpichov created an issue 
 
 
 
 
 
 
 
 
 
 

 
 
 
 
 
 
 
 Beam /  BEAM-1354 
 
 
 
  TextIO should comply with PTransform style guide  
 
 
 
 
 
 
 
 
 

Issue Type:
 
  Bug 
 
 
 

Assignee:
 
 Davor Bonaci 
 
 
 

Components:
 

 sdk-java-core 
 
 
 

Created:
 

 31/Jan/17 01:46 
 
 
 

Priority:
 
  Major 
 
 
 

Reporter:
 
 Eugene Kirpichov 
 
 
 
 
 
 
 
 
 
 
 

TextIO. {Read,Write} 
. {Bound,Unbound} 
 are banned names. Instead, TextIO should make the user specify the type parameters explicitly, and have simply TextIO.Read and TextIO.Write themselves be the transform tyles.
 

Both should take simply String, and not use Coder as a general-purpose serialization mechanism.
 

Should perhaps use AutoValue for parameter builders