fluo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From keith-turner <...@git.apache.org>
Subject [GitHub] incubator-fluo-recipes pull request #128: Updated ExportQ and CFM to use new...
Date Tue, 25 Apr 2017 15:14:29 GMT
Github user keith-turner commented on a diff in the pull request:

    https://github.com/apache/incubator-fluo-recipes/pull/128#discussion_r113224356
  
    --- Diff: docs/accumulo-export-queue.md ---
    @@ -19,8 +19,8 @@ limitations under the License.
     ## Background
     
     The [Export Queue Recipe][1] provides a generic foundation for building export mechanism
to any
    -external data store. The [AccumuloExporter] provides an implementation of this recipe
for
    -Accumulo. The [AccumuloExporter] is located the `fluo-recipes-accumulo` module and provides
the
    +external data store. The [AccumuloConsumer] provides an export consumer for writing to
    --- End diff --
    
    A bit of background.  The name I like most is AccumuloExporter. I like this name because
it implies sending things out from Fluo.  However this name is currently taken by a class
that's deprecated in this PR.
    
    The super type for AccumuloConsumer is ExportConsumer (which is a new type in this PR).
 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message