leosanqing opened a new pull request, #4681:
URL: https://github.com/apache/incubator-inlong/pull/4681

   
   ### Prepare a Pull Request
   
   - Fixes #4674 
   
   ### Motivation
   
   The previous implementation of the client is not particularly good
   1. The authentication information of the user's login is spliced ​​in the 
path each time
   2. Param and PathVariable are manually spliced ​​together, and the 
scalability is not good
   3. All the request methods for connecting to the web are all in one class
   4. The previous execution of the http request package is not concise enough
   
   ### Modifications
   
   1. The processing of authentication information is placed in the 
interceptor, and then if the authentication information is replaced by a 
header, it only needs to be modified here
   2. The method of client requesting web controller is divided according to 
business, corresponding to the web aspect, which is convenient for maintenance 
and search
   3. Use declarative annotations to simplify development and maintenance
   
   ### Verifying this change
   
   *(Please pick either of the following options)*
   
   - [x] This change is already covered by existing tests, such as:
     - Kafka2HiveExample
     - InnerInlongManagerClientTest
   
   


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