Skip to content
This repository was archived by the owner on Mar 28, 2025. It is now read-only.

Conversation

@amoldabade7
Copy link

No description provided.

@amoldabade7 amoldabade7 force-pushed the try-es-increased-timeout branch from 3b4a672 to 6dbdf3c Compare April 18, 2022 15:18
@amoldabade7 amoldabade7 force-pushed the try-es-increased-timeout branch from fbdc6f7 to 74712ac Compare April 19, 2022 15:34
@bobglobe
Copy link

CI is failing

options().getIndex(),
DOCUMENT_TYPE);

config = config.withSocketTimeout(1800000);

Choose a reason for hiding this comment

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

Shall we make this a parameter instead of hardcoding it ?

Copy link
Author

Choose a reason for hiding this comment

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

It's good idea.

Copy link
Author

Choose a reason for hiding this comment

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

Moved to a configurable value for socketTimeout value.

@amoldabade7
Copy link
Author

Failing CI checks are for irrelevant repos (that are not modified in the PR)

@amoldabade7 amoldabade7 requested a review from bobglobe April 21, 2022 06:28
tanguycdls pushed a commit that referenced this pull request Jul 27, 2023
* Spanner VertexAI Integaration: Add Model DDL.

* Spanner VertexAI Integaration: Add Model DDL.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants