yes they would. I will modify the tests to work directly off of
traversablespout
and send a PR


On Wed, Sep 25, 2013 at 9:46 AM, Sam Ritchie <notifications@github.com>wrote:

> In
> tormenta-avro/src/main/scala/com/twitter/tormenta/spout/avro/AvroTraversableSpout.scala:
>
> > + *    distributed under the License is distributed on an "AS IS" BASIS,
> > + *    WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
> > + *    See the License for the specific language governing permissions and
> > + *    limitations under the License.
> > + */
> > +package com.twitter.tormenta.spout.avro
> > +
> > +import com.twitter.tormenta.spout.TraversableSpout
> > +import org.apache.avro.generic.GenericRecord
> > +import org.apache.avro.specific.SpecificRecordBase
> > +
> > +/**
> > + * @author Mansur Ashraf
> > + * @since 9/24/13
> > + */
> > +class GenericRecordTraversableSpout(items: TraversableOnce[GenericRecord], fieldName: String) extends TraversableSpout[GenericRecord](items, fieldName)
>
> I guess what I'm wondering is, why do we need a specialized type? I think
> the tests you added should all pass with the basic traversablespout, right?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/twitter/tormenta/pull/43/files#r6574305>
> .
>