Skip to content
This repository was archived by the owner on Jun 4, 2024. It is now read-only.

Event-handler: call SearchUnstructuredEvents with smaller windows#1068

Open
tigrato wants to merge 6 commits into
masterfrom
tigrato/fix-search-events
Open

Event-handler: call SearchUnstructuredEvents with smaller windows#1068
tigrato wants to merge 6 commits into
masterfrom
tigrato/fix-search-events

Conversation

@tigrato

@tigrato tigrato commented May 13, 2024

Copy link
Copy Markdown
Contributor

This PR aims to reduce the windows sent from SearchUnstructuredEvents. StartTime is never updated so it's kept with its initial value which causes problems to the event handler sending windows that include more than 1Gb of data when using the Athena backend which causes failures.

@tigrato tigrato force-pushed the tigrato/fix-search-events branch from 02ab246 to 4ac84da Compare May 13, 2024 14:40
@tigrato tigrato force-pushed the tigrato/fix-search-events branch 2 times, most recently from 5badd10 to c911475 Compare May 13, 2024 15:05
@tigrato tigrato force-pushed the tigrato/fix-search-events branch from c911475 to 4301b26 Compare May 13, 2024 15:07
@tigrato tigrato force-pushed the tigrato/fix-search-events branch from 4301b26 to 0d3e712 Compare May 13, 2024 15:14
@tigrato tigrato force-pushed the tigrato/fix-search-events branch from 0d3e712 to 68aeb02 Compare May 13, 2024 15:18
This PR aims to reduce the windows sent from `SearchUnstructuredEvents`.
StartTime is never update so it's kept with its initial value which
causes problems to the event handler sending windows that include more
than 1Gb of data when using the Athena backend which causes failures.

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>
@tigrato tigrato force-pushed the tigrato/fix-search-events branch from 68aeb02 to 37db668 Compare May 13, 2024 15:45
@tigrato tigrato marked this pull request as ready for review May 13, 2024 16:38
@tigrato tigrato force-pushed the tigrato/fix-search-events branch from c78062c to b413446 Compare May 14, 2024 13:50
@tigrato tigrato force-pushed the tigrato/fix-search-events branch from f587ade to 56e1614 Compare May 14, 2024 14:59
tigrato added a commit to gravitational/teleport that referenced this pull request May 17, 2024
This PR aims to reduce the windows sent from `SearchUnstructuredEvents`. StartTime is never updated so it's kept with its initial value which causes problems to the event handler sending windows that include more than 1Gb of data when using the Athena backend which causes failures.

Syncs with gravitational/teleport-plugins#1068
Related to #41544

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>
tigrato added a commit to gravitational/teleport that referenced this pull request May 18, 2024
This PR aims to reduce the windows sent from `SearchUnstructuredEvents`. StartTime is never updated so it's kept with its initial value which causes problems to the event handler sending windows that include more than 1Gb of data when using the Athena backend which causes failures.

Syncs with gravitational/teleport-plugins#1068
Related to #41544

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>
github-merge-queue Bot pushed a commit to gravitational/teleport that referenced this pull request May 21, 2024
…41730)

* Event-handler: call `SearchUnstructuredEvents` with smaller windows

This PR aims to reduce the windows sent from `SearchUnstructuredEvents`. StartTime is never updated so it's kept with its initial value which causes problems to the event handler sending windows that include more than 1Gb of data when using the Athena backend which causes failures.

Syncs with gravitational/teleport-plugins#1068
Related to #41544

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>

* fix lint

* add configurable window size

---------

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>
github-actions Bot pushed a commit to gravitational/teleport that referenced this pull request May 21, 2024
This PR aims to reduce the windows sent from `SearchUnstructuredEvents`. StartTime is never updated so it's kept with its initial value which causes problems to the event handler sending windows that include more than 1Gb of data when using the Athena backend which causes failures.

Syncs with gravitational/teleport-plugins#1068
Related to #41544

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>
github-merge-queue Bot pushed a commit to gravitational/teleport that referenced this pull request May 22, 2024
…dows (#41904)

* Event-handler: call `SearchUnstructuredEvents` with smaller windows

This PR aims to reduce the windows sent from `SearchUnstructuredEvents`. StartTime is never updated so it's kept with its initial value which causes problems to the event handler sending windows that include more than 1Gb of data when using the Athena backend which causes failures.

Syncs with gravitational/teleport-plugins#1068
Related to #41544

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>

* fix lint

* add configurable window size

---------

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>
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