sungwy opened a new pull request, #2766:
URL: https://github.com/apache/iceberg-python/pull/2766

   <!--
   Thanks for opening a pull request!
   -->
   
   <!-- In the case this PR will resolve an issue, please replace 
${GITHUB_ISSUE_ID} below with the actual Github issue id. -->
   <!-- Closes #${GITHUB_ISSUE_ID} -->
   
   # Rationale for this change
   
   Reduce unintentionally exposed public surface. Related issue: 
https://github.com/apache/iceberg-python/issues/1099
   
   ## Are these changes tested?
   
   ## Are there any user-facing changes?
   
   Yes - and intentionally so. The number of public APIs exposed through the 
avro modules will be reduced significantly.
   
   A quick search through 
[Daft](https://github.com/search?q=repo%3AEventual-Inc%2FDaft%20%22pyiceberg.avro%22&type=code)
 and 
[DuckDB](https://github.com/search?q=repo%3Aduckdb%2Fduckdb%20%22pyiceberg.avro%22&type=code)
 showed no direct usage of `pyiceberg.avro` module and its child modules in 
their non-test code.
   
   TODO:
   - [ ] I'll generate the stats of public API numbers before and after this 
change and add it to this PR
   - [ ] Changelog
   
   
   <!-- In the case of user-facing changes, please add the changelog label. -->
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to