Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove GemFire connector #1561

Merged
merged 15 commits into from
Jun 26, 2021
Merged

Remove GemFire connector #1561

merged 15 commits into from
Jun 26, 2021

Conversation

sumwale
Copy link
Contributor

@sumwale sumwale commented Jun 26, 2021

For the upcoming open-sourcing of snappy-connectors repository, the gemfire-connector module
still depends on the closed-source Pivotal's Gemfire jar, so removed those references.

It can be added back in future if the code in snappy-connectors repository is
modified to support Apache Geode instead.

Also added OSS versions of snappy-aqp and snappy-connectors repositories as submodules.

Upgraded gradle to v5.6.4, the latest 5.x release.

Fixed multiple test failures.

sumwale and others added 15 commits June 8, 2021 17:22
For the upcoming open-sourcing of snappy-connectors repository, the gemfire-connector module
still depends on the closed-source Pivotal's Gemfire jar, so removed those references.

It can be added back in future if the code in snappy-connectors repository is
modified to support Apache Geode instead.
also restoring code to build gemfire-connector only if directory is present
also moved common test cluster start/stop etc to new class ClusterUtils that also
creates a separate directory for cluster instead of using the main snappy product directory
- fix snappy-nodes.sh script to work uniformly for both local and remote execution
- updated year in LICENSE
- also fixed occasional bulk write lock timeout
- updated submodule links
@sumwale sumwale self-assigned this Jun 26, 2021
@sumwale sumwale merged commit 8c7c02e into master Jun 26, 2021
@sumwale sumwale deleted the remove-gfeconnector branch June 26, 2021 03:31
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.

1 participant