Closed YashJipkate closed 3 years ago
Thanks for opening a pull request!
If this is not a minor PR. Could you open an issue for this pull request on JIRA? https://issues.apache.org/jira/browse/ARROW
Opening JIRAs ahead of time contributes to the Openness of the Apache Arrow project.
Then could you also rename pull request title in the following format?
ARROW-${JIRA_ID}: [${COMPONENT}] ${SUMMARY}
or
MINOR: [${COMPONENT}] ${SUMMARY}
See also:
@YashJipkate The changes look great! I left some comments.
@JayjeetAtGithub I have done the requested changes, please take a look.
This PR adds docstrings to file_rados_parquet.cc and file_rados_parquet.h files. The docstrings follow the current upstream arrow repo standards. The docstrings still need to be improved and approved before merging.