-
Couldn't load subscription status.
- Fork 5
add version for kafka tag #37
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
base: master
Are you sure you want to change the base?
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.
Just a question any reason not to use the versions file like everything else ?
I've added the kafkatag to the json file, i think first time through I was in "developer mode" to reuse/adapt what was there. But I see adding the value to the json it become much more flexible for us when testing an inflight version. |
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.
Looks good !
Just a few questions though.
| "sparkclient": "repo1.maven.org/maven2/com/ibm/event/ibm-db2-eventstore-client-spark-2.4.6/2.0.1.0/ibm-db2-eventstore-client-spark-2.4.6-2.0.1.0.jar", | ||
| "hadoopversion":"2.6", | ||
| "scopt211version":"3.5.0" | ||
| "scopt211version":"3.5.0", |
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.
I know this was here before but what's this used for ? I didn't see us consuming this for any versioning ?
| "scopt211version":"3.5.0", | ||
| "kafkatag":"v2.0.0.5" | ||
| } , | ||
| "dev" : { |
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.
What's the idea behind this ? Is it just a placeholder to make it easier for devs to make changes going forward ?
basic story requirements futhfilled, will look if we need a subsequent story to expand support for developers using not released versions of ES with Kafka