[KAFKA-246] Support namespaces provided in the Schema Created: 11/Aug/21 Updated: 28/Oct/23 Resolved: 16/Feb/22 |
|
| Status: | Closed |
| Project: | Kafka Connector |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 1.7.0 |
| Type: | Improvement | Priority: | Unknown |
| Reporter: | Ross Lawley | Assignee: | Valentin Kavalenka |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | size-small | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||
| Quarter: | FY22Q4 | ||||
| Case: | (copied to CRM) | ||||
| Documentation Changes: | Needed | ||||
| Description |
|
| Comments |
| Comment by Githook User [ 16/Feb/22 ] |
|
Author: {'name': 'Valentin Kovalenko', 'email': 'valentin.kovalenko@mongodb.com', 'username': 'stIncMale'}Message: Correctly process namespaces when converting `org.apache.avro.Schema` to `org.apache.kafka.connect.data.Schema` (#103) `org.apache.kafka.connect.data.Schema` does not separate a namespace from a short name, Avro supports namespaces only for schemas of records, enums, and fixed types, It is worth pointing out that as a result of this change, MongoDB Kafka Connector
Other useful links:
|
| Comment by Jing Yong Lee [ 29/Nov/21 ] |
|
Hi @Ross Lawley/@Esha Bhargava, could I submit a proposed fix to this issue via a pull request? I think we can include the namespace from org.apache.avro.Schema as a part of org.apache.kafka.connect.data.Schema's name field to ensure the namespace is not omitted from the org.apache.kafka.connect.data.Schema produced by com.mongodb.kafka.connect.source.schema.AvroSchema's createSchema method. This practice is also used in Confluent's io.confluent.connect.avro.AvroData: Thanks. |
| Comment by Ross Lawley [ 19/Aug/21 ] |
|
Hi guptabrijmohan30@gmail.com, I don't think a PR has been created for this feature yet. Its not planned for release this quarter. Regarding the test failure, you'd have to look at the test report to understand more about what failed and why. Ross |
| Comment by BRIJ MOHAN GUPTA [ 19/Aug/21 ] |
|
Hi @Ross Lawley, Please let me know when you are targeting this release? Because I am not able to build given PR. |