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
[GORA-638] Add gora-ignite into the gora-tutorial module #195
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 from me
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1, LGTM
pom.xml
Outdated
@@ -1012,6 +1012,17 @@ | |||
<version>${project.version}</version> | |||
<type>test-jar</type> | |||
</dependency> | |||
<dependency> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<dependency>
<groupId>org.apache.gora</groupId>
<artifactId>gora-redis</artifactId>
<version>${project.version}</version>
</dependency>
<dependency>
<groupId>org.apache.gora</groupId>
<artifactId>gora-redis</artifactId>
<version>${project.version}</version>
<type>test-jar</type>
</dependency>
Do we need these redis dependencies as part of this PR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for that. I unintentionally mixed code from other experiments.
@joseph6x Thank you for your contribution. :) This looks good to me. I will locally test and merge the PR. |
@joseph6x I merged your PR. Thank you for this and looking forward for your future contributions. :) |
Add support for Ignite in the gora-tutorial module.
As part of my experiments with Gora, I tried to add ignite in the tutorial module using other other databases as example. Please let me know if I missed something.