zanmato1984 commented on PR #46210: URL: https://github.com/apache/arrow/pull/46210#issuecomment-2860186791
Hi, @amoeba @raulcd @pitrou , could you help on reviewing this? I'm looking forward to having this merged so I can proceed with the subsequent refinement items. Thanks. > > I also think the doc @amoeba pointed out, > > > ... but we have at least one precedent for such lower-level docs with the Acero [Developer's Guide](https://arrow.apache.org/docs/cpp/acero/developer_guide.html). > > as suggested by @pitrou , > > > I also think those should not end up in the user's guide. > > could be moved along too, probably in a following PR? > > And for the [Swiss table doc](https://github.com/apache/arrow/blob/main/cpp/src/arrow/acero/doc/key_map.md) I was initially referencing too, I'll do the same to it as this row table doc. > > And maybe we can further add more developer-oriented doc, esp. for compute and Acero, in the future. Because I found they deserve better documentation. > > Thoughts? -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
