You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@samza.apache.org by Eli Reisman <in...@gmail.com> on 2015/08/07 01:23:39 UTC

Re: Review Request 35445: SAMZA-693: Very basic HDFS Producer service for Samza

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35445/
-----------------------------------------------------------

(Updated Aug. 6, 2015, 11:23 p.m.)


Review request for samza.


Repository: samza


Description
-------

SAMZA-693: Very basic HDFS Producer service for Samza


Diffs (updated)
-----

  build.gradle 0852adc 
  docs/learn/documentation/versioned/hdfs/producer.md PRE-CREATION 
  docs/learn/documentation/versioned/index.html 85209bd 
  docs/learn/documentation/versioned/jobs/configuration-table.html 926fd50 
  samza-hdfs/src/main/scala/org/apache/samza/system/hdfs/HdfsConfig.scala PRE-CREATION 
  samza-hdfs/src/main/scala/org/apache/samza/system/hdfs/HdfsSystemAdmin.scala PRE-CREATION 
  samza-hdfs/src/main/scala/org/apache/samza/system/hdfs/HdfsSystemFactory.scala PRE-CREATION 
  samza-hdfs/src/main/scala/org/apache/samza/system/hdfs/HdfsSystemProducer.scala PRE-CREATION 
  samza-hdfs/src/main/scala/org/apache/samza/system/hdfs/HdfsSystemProducerMetrics.scala PRE-CREATION 
  samza-hdfs/src/main/scala/org/apache/samza/system/hdfs/writer/BinarySequenceFileHdfsWriter.scala PRE-CREATION 
  samza-hdfs/src/main/scala/org/apache/samza/system/hdfs/writer/Bucketer.scala PRE-CREATION 
  samza-hdfs/src/main/scala/org/apache/samza/system/hdfs/writer/HdfsWriter.scala PRE-CREATION 
  samza-hdfs/src/main/scala/org/apache/samza/system/hdfs/writer/JobNameDateTimeBucketer.scala PRE-CREATION 
  samza-hdfs/src/main/scala/org/apache/samza/system/hdfs/writer/SequenceFileHdfsWriter.scala PRE-CREATION 
  samza-hdfs/src/main/scala/org/apache/samza/system/hdfs/writer/TextSequenceFileHdfsWriter.scala PRE-CREATION 
  samza-hdfs/src/test/resources/samza-hdfs-test-batch-job-text.properties PRE-CREATION 
  samza-hdfs/src/test/resources/samza-hdfs-test-batch-job.properties PRE-CREATION 
  samza-hdfs/src/test/resources/samza-hdfs-test-job-text.properties PRE-CREATION 
  samza-hdfs/src/test/resources/samza-hdfs-test-job.properties PRE-CREATION 
  samza-hdfs/src/test/scala/org/apache/samza/system/hdfs/TestHdfsSystemProducerTestSuite.scala PRE-CREATION 
  settings.gradle 19bff97 

Diff: https://reviews.apache.org/r/35445/diff/


Testing
-------

Updated: See JIRA SAMZA-693 for details, this latest update (693-6) should be good to go. Thanks again for the execellent reviews and prompt responses, your time is appreciated!

Passes 'gradle clean test'.


Thanks,

Eli Reisman