-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Downloading specific version of schemas capability added #2178
Conversation
Thanks @pagrawal10 , the new functionality should be backward compatible. Won't this break users who are not specifying any versions? |
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.
Left some more comments. Also can you add unit tests DownloadSchemaRegistryMojoTest
?
...plugin/src/main/java/io/confluent/kafka/schemaregistry/maven/DownloadSchemaRegistryMojo.java
Show resolved
Hide resolved
...plugin/src/main/java/io/confluent/kafka/schemaregistry/maven/DownloadSchemaRegistryMojo.java
Show resolved
Hide resolved
...plugin/src/main/java/io/confluent/kafka/schemaregistry/maven/DownloadSchemaRegistryMojo.java
Show resolved
Hide resolved
...plugin/src/main/java/io/confluent/kafka/schemaregistry/maven/DownloadSchemaRegistryMojo.java
Show resolved
Hide resolved
...plugin/src/main/java/io/confluent/kafka/schemaregistry/maven/DownloadSchemaRegistryMojo.java
Show resolved
Hide resolved
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.
One minor nit, otherwise LGTM
Objective : To download a specific version associated with a subjectPattern.
To run:
Parameters to be passed while using this goal :
schemaRegistryUrls
Type : List
Required = true
outputDirectory
Type : File
Required = true
versions
Type : List
Required = true
subjectPatterns
Type : List
Required = true