Skip to content

Add setters to VectorOverrideCommandLineOptions#1281

Merged
rfecher merged 1 commit intolocationtech:masterfrom
ahuarte47:master-landsat8-vectoroptions
Mar 1, 2018
Merged

Add setters to VectorOverrideCommandLineOptions#1281
rfecher merged 1 commit intolocationtech:masterfrom
ahuarte47:master-landsat8-vectoroptions

Conversation

@ahuarte47
Copy link
Copy Markdown
Contributor

@ahuarte47 ahuarte47 commented Mar 1, 2018

This PR adds the 'vectorStore' and 'vectorIndex' setters of VectorOverrideCommandLineOptions in Landsat8 client module. They can not be configured now.

@ahuarte47 ahuarte47 force-pushed the master-landsat8-vectoroptions branch from f993bd5 to 417ca1b Compare March 1, 2018 11:29
@rfecher
Copy link
Copy Markdown
Contributor

rfecher commented Mar 1, 2018

@ahuarte47 Have you signed an Eclipse Contributor Agreement? Its electronic and it is a requirement of Eclipse for new contributors to GeoWave to sign this: https://www.eclipse.org/contribute/cla

The contribution looks applicable, and we appreciate the test updates as well! Also, I saw what you were able to do with the EEA data - very cool!

@ahuarte47
Copy link
Copy Markdown
Contributor Author

Hi @rfecher, thank you very much! I have just signed the ECA

@rfecher rfecher merged commit e593b6c into locationtech:master Mar 1, 2018
@ahuarte47
Copy link
Copy Markdown
Contributor Author

Thanks @rfecher for reviewing and merging

@ahuarte47 ahuarte47 deleted the master-landsat8-vectoroptions branch March 1, 2018 22:38
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