Skip to content

fix(ingest/aerospike): pass socket_timeout via query.results() policy#17584

Open
dotan-mor wants to merge 9 commits into
datahub-project:masterfrom
dotan-mor:fix/aerospike-socket-timeout
Open

fix(ingest/aerospike): pass socket_timeout via query.results() policy#17584
dotan-mor wants to merge 9 commits into
datahub-project:masterfrom
dotan-mor:fix/aerospike-socket-timeout

Conversation

@dotan-mor
Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot added ingestion PR or Issue related to the ingestion of metadata community-contribution PR or Issue raised by member(s) of DataHub Community labels May 26, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 26, 2026

Bundle Report

Bundle size has no change ✅

@maggiehays maggiehays added the needs-review Label for PRs that need review from a maintainer. label May 26, 2026
@dotan-mor dotan-mor changed the title Fix/aerospike socket timeout fix(ingest/aerospike): pass socket_timeout via query.results() policy May 26, 2026
Comment thread metadata-ingestion/src/datahub/ingestion/source/aerospike.py
@rajatoss
Copy link
Copy Markdown
Member

I have reviewed it, if possible please check the tests you have added for duplicate nature, I think some of them can be removed or at-least combined

@maggiehays maggiehays added pending-submitter-response Issue/request has been reviewed but requires a response from the submitter and removed needs-review Label for PRs that need review from a maintainer. labels May 28, 2026
@maggiehays maggiehays added needs-review Label for PRs that need review from a maintainer. and removed pending-submitter-response Issue/request has been reviewed but requires a response from the submitter labels May 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-contribution PR or Issue raised by member(s) of DataHub Community ingestion PR or Issue related to the ingestion of metadata needs-review Label for PRs that need review from a maintainer.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants