Skip to content

Conversation

@luohoufu
Copy link
Contributor

@luohoufu luohoufu commented Nov 28, 2023

Purpose of this pull request

Support INFINI Easysearch

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

Copy link
Member

@hailin0 hailin0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please check ci errors

@luohoufu
Copy link
Contributor Author

Please check ci errors

Build Successful

@@ -0,0 +1,178 @@
# INFINI Easysearch
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -0,0 +1,198 @@
# Easysearch
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

query = {"range": {"c_int": {"gte": 10, "lte": 20}}}
schema = {
fields {
c_map = "map<string, tinyint>"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please test read all datatypes of source

@hailin0 hailin0 added this to the 2.3.4 milestone Nov 29, 2023
Copy link
Contributor

@lightzhao lightzhao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hailin0 hailin0 added the no update The owner doesn't provide further feedback. label Dec 11, 2023
hailin0
hailin0 previously approved these changes Dec 14, 2023

Engine Supported

* Supported all versions released by [INFINI Easysearch](https://www.infini.com/download/?product=easysearch).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

Engine Supported lost.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated

* Supported all versions released by [INFINI Easysearch](https://www.infini.com/download/?product=easysearch).

:::

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated

## Description

Used to read data from INFINI Easysearch.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated

- [x] [column projection](../../concept/connector-v2-features.md)
- [ ] [parallelism](../../concept/connector-v2-features.md)
- [ ] [support user-defined split](../../concept/connector-v2-features.md)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@EricJoy2048 EricJoy2048 removed this from the 2.3.4 milestone Jan 5, 2024
EricJoy2048
EricJoy2048 previously approved these changes Jan 18, 2024
@luohoufu luohoufu dismissed stale reviews from EricJoy2048 and hailin0 via 2076cfc January 24, 2024 01:21
@github-actions github-actions bot removed the approved label Jan 29, 2024
@hailin0 hailin0 merged commit 41e6288 into apache:dev Mar 6, 2024
chaorongzhi pushed a commit to chaorongzhi/seatunnel that referenced this pull request Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved connectors-v2 feature New feature no update The owner doesn't provide further feedback. reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants