You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2021/01/29 04:43:07 UTC

[GitHub] [arrow] zhztheplayer edited a comment on pull request #7030: ARROW-7808: [Java][Dataset] Implement Dataset Java API by JNI to C++

zhztheplayer edited a comment on pull request #7030:
URL: https://github.com/apache/arrow/pull/7030#issuecomment-766616108


   Hi @emkornfield @pitrou 
   
   I think I have addressed existing comments so would you like to take another look? Thanks a lot.
   
   Based on some recent comments I think we may don't have to make 100% consensus on the current design of `ReservationListenableMemoryPool` or relevant things instantly, if this part is in a way controversial I could split out the part from this PR then create a new one for it if we can possibly get this merged.
   
   And if you have other suggestions on the whole design or architecture of the basic functionality please be free to add comments and I am happy to make further refactors accordingly. 


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org