datahub icon indicating copy to clipboard operation
datahub copied to clipboard

Remove confluent schema registry dependency

Open tmnd1991 opened this issue 2 years ago • 3 comments

Checklist

  • [ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
  • [ ] Links to related issues (if applicable)
  • [ ] Tests for the changes have been added/updated (if applicable)
  • [ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
  • [ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

tmnd1991 avatar Jun 23 '22 07:06 tmnd1991

Unit Test Results (build & test)

0 tests   - 389   0 :heavy_check_mark:  - 389   0s :stopwatch: - 11m 21s 0 suites  -   92   0 :zzz: ±    0  0 files    -   92   0 :x: ±    0 

Results for commit 767d19c2. ± Comparison against base commit 9f2b3b9b.

github-actions[bot] avatar Jun 23 '22 18:06 github-actions[bot]

Unit Test Results (metadata ingestion)

       8 files  ±0         8 suites  ±0   1h 15m 59s :stopwatch: - 2m 47s    576 tests ±0     573 :heavy_check_mark: +2    3 :zzz: ±0  0 :x:  - 2  1 090 runs  ±0  1 047 :heavy_check_mark: +2  43 :zzz: ±0  0 :x:  - 2 

Results for commit 767d19c2. ± Comparison against base commit 9f2b3b9b.

github-actions[bot] avatar Jun 23 '22 18:06 github-actions[bot]

Any movement on this PR? I am really looking forward to it.

hendrix04 avatar Jul 29 '22 19:07 hendrix04

Hi @tmnd1991 !

Thanks for this contribution. We are going to close this PR until further notice. This is a great start obviously but there's much more work to do to fully remove the registry.

We're still very much interested in removing the registry, so feel free to reopen if you've got time to work on it again!

John

jjoyce0510 avatar Nov 17 '22 05:11 jjoyce0510