Skip to content

Conversation

@robertpangborn
Copy link
Collaborator

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

Copy link
Collaborator

@hroumani hroumani left a 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 ?

@robertpangborn
Copy link
Collaborator Author

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.

Copy link
Collaborator

@hroumani hroumani left a 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",
Copy link
Collaborator

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" : {
Copy link
Collaborator

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 ?

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