Skip to content

GEOMESA-724,GEOMESA-729 Map/reduce and scalding input and output formats#511

Closed
elahrvivaz wants to merge 5 commits into
locationtech:accumulo1.5.x/1.xfrom
elahrvivaz:fcr_geomesa_input_output
Closed

GEOMESA-724,GEOMESA-729 Map/reduce and scalding input and output formats#511
elahrvivaz wants to merge 5 commits into
locationtech:accumulo1.5.x/1.xfrom
elahrvivaz:fcr_geomesa_input_output

Conversation

@elahrvivaz

Copy link
Copy Markdown
Contributor
  • Split query planning and query execution into two separate steps so that m/r can use query planning
  • Split feature writing into creating mutations and applying them so that m/r can use the mutations directly
  • Input - can specify CQL which will be translated into the appropriate ranges and iterators behind the scenes
  • Output - can write features directly to accumulo
  • Input and output delegate to AccumuloInput/OutputFormat
  • Supports 'old' and 'new' hadoop apis (mapred vs mapreduce packages)
  • Provides hadoop serialization for simple features
  • Split AccumuloSource options into input and output specific flavors
  • For interoperability, has Java implementations that call to the scala classes
  • Sample jobs are included - FeatureCountJob and FeatureWriterJob

Signed-off-by: Emilio Lahr-Vivaz elahrvivaz@ccri.com

@elahrvivaz elahrvivaz force-pushed the fcr_geomesa_input_output branch from 76b3355 to e03df3e Compare March 24, 2015 22:36
@elahrvivaz elahrvivaz changed the title GEOMESA-724 Map/reduce and scalding input and output formats GEOMESA-724,GEOMESA-729 Map/reduce and scalding input and output formats Mar 24, 2015

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this break backward compatibility with version 0?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, but it's already broken due to schema_changes, with no upgrade path available... Is that something we need to worry about?

@jnh5y

jnh5y commented Mar 24, 2015

Copy link
Copy Markdown
Contributor

Looks great. Most of my comments are small/quick. I'd like to get another +1.

@jnh5y jnh5y force-pushed the accumulo1.5.x/1.x branch 2 times, most recently from efa705f to 969513a Compare March 25, 2015 21:06
@elahrvivaz elahrvivaz force-pushed the fcr_geomesa_input_output branch from 61aa59f to aaeff22 Compare March 26, 2015 12:54
* Split query planning and query execution into two separate steps so that m/r can use query planning
* Split feature writing into creating mutations and applying them so that m/r can use the mutations directly
* Input - can specify CQL which will be translated into the appropriate ranges and iterators behind the scenes
* Output - can write features directly to accumulo
* Input and output delegate to AccumuloInput/OutputFormat
* Supports 'old' and 'new' hadoop apis (mapred vs mapreduce packages)
* Provides hadoop serialization for simple features
* Split AccumuloSource options into input and output specific flavors
* For interoperability, has Java implementations that call to the scala classes
* Sample jobs are included - FeatureCountJob and FeatureWriterJob

Signed-off-by: Emilio Lahr-Vivaz <elahrvivaz@ccri.com>
Signed-off-by: Emilio Lahr-Vivaz <elahrvivaz@ccri.com>
Signed-off-by: Emilio Lahr-Vivaz <elahrvivaz@ccri.com>
@elahrvivaz elahrvivaz force-pushed the fcr_geomesa_input_output branch from 6f61ded to 22a57f5 Compare March 26, 2015 14:13
Signed-off-by: Emilio Lahr-Vivaz <elahrvivaz@ccri.com>
@elahrvivaz elahrvivaz force-pushed the fcr_geomesa_input_output branch from ffe8822 to a0401d4 Compare March 27, 2015 18:52
@jnh5y

jnh5y commented Mar 31, 2015

Copy link
Copy Markdown
Contributor

@anthonyccri @elahrvivaz This got merged a-ok, right? I'm just curious why it is still open. If no one gets to it first, I can sort it out in the morning.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants