Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Improved README #735

Merged
merged 1 commit into from
Jan 8, 2022
Merged

Improved README #735

merged 1 commit into from
Jan 8, 2022

Conversation

jorgecarleitao
Copy link
Owner

  • Added enumeration of current features
  • removed section with discussion on why Arrow API has issues (it belongs somewhere else)
  • Removed section with feature-comparison it with arrow (it tends to get outdated from both ends)
  • added section with our safety and security practices and main associated risks
  • Expanded section on what is being tested for interoperability

@jorgecarleitao jorgecarleitao added the documentation Improvements or additions to documentation label Jan 6, 2022
@jorgecarleitao
Copy link
Owner Author

@houqp @Dandandan @ritchie46 @sundy-li , could you assess whether the new wording is balanced?

@codecov
Copy link

codecov bot commented Jan 6, 2022

Codecov Report

Merging #735 (668dad9) into main (c28d69e) will increase coverage by 0.38%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #735      +/-   ##
==========================================
+ Coverage   70.53%   70.91%   +0.38%     
==========================================
  Files         311      311              
  Lines       16920    17355     +435     
==========================================
+ Hits        11934    12308     +374     
- Misses       4986     5047      +61     
Impacted Files Coverage Δ
src/io/avro/read/deserialize.rs 66.47% <0.00%> (-0.48%) ⬇️
src/io/ipc/write/schema.rs 92.44% <0.00%> (-0.34%) ⬇️
src/io/ipc/read/reader.rs 72.18% <0.00%> (-0.28%) ⬇️
benches/filter_kernels.rs 0.00% <0.00%> (ø)
src/datatypes/mod.rs 98.11% <0.00%> (+0.89%) ⬆️
src/datatypes/schema.rs 25.00% <0.00%> (+1.00%) ⬆️
src/io/json_integration/read/schema.rs 65.24% <0.00%> (+1.24%) ⬆️
src/io/parquet/read/schema/convert.rs 84.61% <0.00%> (+6.64%) ⬆️
src/datatypes/field.rs 36.20% <0.00%> (+8.75%) ⬆️
src/io/ipc/read/schema.rs 90.13% <0.00%> (+9.48%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c28d69e...668dad9. Read the comment docs.

@sundy-li
Copy link
Collaborator

sundy-li commented Jan 7, 2022

/LGTM

@jorgecarleitao jorgecarleitao merged commit 772ef0f into main Jan 8, 2022
@jorgecarleitao jorgecarleitao deleted the readme branch January 8, 2022 09:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants