Skip to content

[INLONG-7220][Manager] Optimize OpenStreamSinkController implementation#7224

Merged
gosonzhang merged 2 commits intoapache:masterfrom
gosonzhang:INLONG-7220
Jan 12, 2023
Merged

[INLONG-7220][Manager] Optimize OpenStreamSinkController implementation#7224
gosonzhang merged 2 commits intoapache:masterfrom
gosonzhang:INLONG-7220

Conversation

@gosonzhang
Copy link
Contributor

Modifications:

  1. Most of them add error codes to the exceptions thrown, adjust the logs, including removing the printed logs with clear exceptions, and adjust the log level to debug for the path logs;
  2. Adjust the field requirements of the SinkRequest request, and limit the mandatory fields for id-based and key-based updates respectively;
  3. Modify 2 Bugs:
    a. For the operation modified by id, the groupId and streamId are not allowed to be modified in the open api:
    image

b. Correct the judgment on line 93 of HudiSinkOperator.java
image

@gosonzhang gosonzhang merged commit ff68b8d into apache:master Jan 12, 2023
@gosonzhang gosonzhang deleted the INLONG-7220 branch January 12, 2023 08:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Improve][Manager] Optimize OpenStreamSinkController implementation

4 participants