alkis commented on code in PR #42174:
URL: https://github.com/apache/arrow/pull/42174#discussion_r1683056384


##########
cpp/tools/parquet/CMakeLists.txt:
##########
@@ -32,5 +32,16 @@ if(PARQUET_BUILD_EXECUTABLES)
             RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR})
   endforeach(TOOL)
 
+  # Only build parquet-dump-footer when statically linking.

Review Comment:
   Btw I am trying this and it seems that the main issue is that `FileMetaData` 
is not exposed. Then this becomes pretty inconvenient? `Scrub` can't be a 
public API because it takes a pointer to a thing that is not exported. Plus I 
would need to add a serialization API to binary and one for json.
   
   Maybe I am missing something? If you could sketch out how this will look I 
can give it a shot.



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

Reply via email to