Skip to content

[INLONG-7557][Sort] Fix class incompatible error between elasticsearch6 with elasticsearch7#7558

Merged
dockerzhang merged 5 commits intoapache:masterfrom
yunqingmoswu:INLONG-7557
Mar 15, 2023
Merged

[INLONG-7557][Sort] Fix class incompatible error between elasticsearch6 with elasticsearch7#7558
dockerzhang merged 5 commits intoapache:masterfrom
yunqingmoswu:INLONG-7557

Conversation

@yunqingmoswu
Copy link
Contributor

Prepare a Pull Request

(Change the title refer to the following example)

Title Example: [INLONG-7557][Sort] Fix class incompatible error between elasticsearch6 with elasticsearch7

(The following XYZ should be replaced by the actual GitHub Issue number)

Motivation

Fix class incompatible error between elasticsearch6 with elasticsearch7 when dirty data archiving is enabled.

Modifications

1.Reconstruct elasticsearch-base and remove the dependency on elasticsearch6.
2.Reconstruct dirty data archiving, add a new DirtySinkFailureHandler to implement dirty data archiving.

Verifying this change

(Please pick either of the following options)

  • 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? (yes / no)
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
  • If a feature is not applicable for documentation, explain why?
  • If a feature is not documented yet in this PR, please create a follow-up issue for adding the documentation

@dockerzhang
Copy link
Contributor

@yunqingmoswu please update the license file and documents if needed.

@dockerzhang dockerzhang merged commit 180b778 into apache:master Mar 15, 2023
Yizhou-Yang pushed a commit to Yizhou-Yang/inlong-yyz that referenced this pull request May 30, 2023
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.

[Bug][Sort] Fix class incompatible error between elasticsearch6 with elasticsearch7

5 participants