@chamikaramj PTAL, I've done another round of updates and addressed all 
comments except the following:
1. Updated time partitioning Java snippet. This will require a PR to the beam 
repo, I opened a JIRA for this update and noted it in the snippet tab. I think 
it's worth publishing the page even without this and then can add the snippet 
in a followup PR.
2. About @jkff 's comment about making sure the examples follow our best 
practices, do you know if these examples are good enough to leave in the page? 
if not, I can remove the section. it seems like some examples are better than 
no examples though.


[ Full content available at: https://github.com/apache/beam-site/pull/376 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to