yfsn666 opened a new pull request, #10807:
URL: https://github.com/apache/inlong/pull/10807

   <!-- Prepare a Pull Request
   Change the title of pull request refer to the following example:
     [INLONG-XYZ][Component] Title of the pull request 
   -->
   
   <!-- Specify the issue this pull request going to fix.
   The following *XYZ* should be replaced by the actual [GitHub 
Issue](https://github.com/apache/inlong/issues) number)-->
   
   Fixes #10806
   
   ### Motivation
   
   <!--Explain here the context, and why you're making that change. What is the 
problem you're trying to solve.-->
   Optimize the python dataproxy sdk build script with the following 
optimizations:
   - Enable user to customize the dynamic repository directory in the build 
script process for the convenience of virtual environment users.
   - The pybind11 folder is not deleted at the end of the build, the build 
process automatically detects if there is a compiled Pybind, and if there is, 
the Pybind download and build steps will be skipped.
   - Do not build dataproxy-cpp-sdk in the python sdk build script, to avoid 
some potential problems, in the build process to automatically detect whether 
the c++ sdk has been compiled. If not compiled, it will provide the compilation 
commands to guide the user to compile the c++ sdk.
   - Remove Pybind test step for faster build.
   
   ### Modifications
   
   <!--Describe the modifications you've done.-->
   Modify the python sdk build script and the correspoding README.
   
   ### Verifying this change
   
   *(Please pick either of the following options)*
   
   - [x] This change is a trivial rework/code cleanup without any test coverage.
   
   - [ ] This change is already covered by existing tests, such as:
     *(please describe tests)*
   
   - [ ] This change added tests and can be verified as follows:
   
     *(example:)*
     - *Added integration tests for end-to-end deployment with large payloads 
(10MB)*
     - *Extended integration test for recovery after broker failure*
   
   ### Documentation
   
     - Does this pull request introduce a new feature? no
   


-- 
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: commits-unsubscr...@inlong.apache.org

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

Reply via email to