hudi-bot opened a new issue, #15757:
URL: https://github.com/apache/hudi/issues/15757

   currently to build hudi-platform-service profile, we need docker or a local 
native thrift binary. see logic related to 
hudi-platform-service/hudi-metaserver/src/main/thrift/bin/thrift_binary.sh
   
   Docker should not be a requirement for build a java project. Some other 
issue: the docker generated source files may lack of permission for current 
user to delete, which could result in mvn clean to fail. encountered this issue 
with GH actions CI. i have to change the ownership for generated folder with 
sudo, but this leaves a loophole in the dev build flow - people may run into 
permission issue when sudo was involved. See this PR 
https://github.com/apache/hudi/pull/7858/
   
   The goal is to remove the docker requirement for build hudi project, and 
make thrift generation smooth across different OSes
   
   
   ## JIRA info
   
   - Link: https://issues.apache.org/jira/browse/HUDI-5708
   - Type: Improvement
   
   
   ---
   
   
   ## Comments
   
   07/Feb/23 15:02;minihippo;Thanks Raymond for reporting and processing this 
issue. I have a proposal about how to load thrift to compile the thrift files 
developer-friendly.
   
   Proposal 1: script for compile and then install  thrift.tar.gz
   
   Proposal 2: provide thrift binary for fixed version OS i.e. centos7, unbantu 
18.04, mac OS
   
   For p1, thrift with a specific version will be installed permanently on a 
developer PC after run once, and the install process cloud have some unexpected 
problem like the lackness of dependencies.
   
   For p2, it has limitations in usage. I'm not sure it can work, I need to 
verify a case.
   
   Or does anyone else have a better suggestion about this?
   
    ;;;


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