Skip to content

Update github actions run conditions #569

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 26, 2025

Conversation

jprakash-db
Copy link
Contributor

Description

Update the conditions on which github actions run, to align with repo development plans

Copy link

Thanks for your contribution! To satisfy the DCO policy in our contributing guide every commit message must include a sign-off message. One or more of your commits is missing this message. You can reword previous commit messages with an interactive rebase (git rebase -i main).

@jprakash-db jprakash-db merged commit eaa542d into main May 26, 2025
23 of 24 checks passed
saishreeeee pushed a commit that referenced this pull request May 26, 2025
More conditions to run github actions
saishreeeee pushed a commit that referenced this pull request May 26, 2025
More conditions to run github actions

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>
varun-edachali-dbx pushed a commit that referenced this pull request May 26, 2025
More conditions to run github actions

Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
varun-edachali-dbx pushed a commit that referenced this pull request May 26, 2025
More conditions to run github actions

Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
varun-edachali-dbx pushed a commit that referenced this pull request May 27, 2025
More conditions to run github actions

Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
varun-edachali-dbx pushed a commit that referenced this pull request May 28, 2025
More conditions to run github actions

Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
varun-edachali-dbx pushed a commit that referenced this pull request May 28, 2025
More conditions to run github actions

Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
saishreeeee added a commit that referenced this pull request May 30, 2025
* PECOBLR-86 Improve logging for debug level

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* PECOBLR-86 Improve logging for debug level

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* fixed format

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* used lazy logging

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* changed debug to error logs

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* added classes required for telemetry

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* removed TelemetryHelper

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* [PECOBLR-361] convert column table to arrow if arrow present (#551)

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* Update CODEOWNERS (#562)

new codeowners

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* Enhance Cursor close handling and context manager exception management to prevent server side resource leaks (#554)

* Enhance Cursor close handling and context manager exception management

* tests

* fmt

* Fix Cursor.close() to properly handle CursorAlreadyClosedError

* Remove specific test message from Cursor.close() error handling

* Improve error handling in connection and cursor context managers to ensure proper closure during exceptions, including KeyboardInterrupt. Add tests for nested cursor management and verify operation closure on server-side errors.

* add

* add

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* PECOBLR-86 improve logging on python driver (#556)

* PECOBLR-86 Improve logging for debug level

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* PECOBLR-86 Improve logging for debug level

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* fixed format

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* used lazy logging

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* changed debug to error logs

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* used lazy logging

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

---------

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* Update github actions run conditions (#569)

More conditions to run github actions

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* Added classes required for telemetry

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* fixed example

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* changed to doc string

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* removed self.telemetry close line

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* grouped classes

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* formatting

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* fixed doc string

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* fixed doc string

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* added more descriptive comments, put dataclasses in a sub-folder

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* fixed default attributes ordering

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* changed file names

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* added enums to models folder

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* removed telemetry batch size

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

---------

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>
Co-authored-by: Shivam Raj <171748731+shivam2680@users.noreply.github.com>
Co-authored-by: Jothi Prakash <jothi.prakash@databricks.com>
Co-authored-by: Madhav Sainanee <madhav.sainanee@databricks.com>
saishreeeee pushed a commit that referenced this pull request Jun 4, 2025
More conditions to run github actions
saishreeeee added a commit that referenced this pull request Jun 4, 2025
* PECOBLR-86 Improve logging for debug level

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* PECOBLR-86 Improve logging for debug level

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* fixed format

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* used lazy logging

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* changed debug to error logs

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* added classes required for telemetry

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* removed TelemetryHelper

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* [PECOBLR-361] convert column table to arrow if arrow present (#551)

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* Update CODEOWNERS (#562)

new codeowners

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* Enhance Cursor close handling and context manager exception management to prevent server side resource leaks (#554)

* Enhance Cursor close handling and context manager exception management

* tests

* fmt

* Fix Cursor.close() to properly handle CursorAlreadyClosedError

* Remove specific test message from Cursor.close() error handling

* Improve error handling in connection and cursor context managers to ensure proper closure during exceptions, including KeyboardInterrupt. Add tests for nested cursor management and verify operation closure on server-side errors.

* add

* add

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* PECOBLR-86 improve logging on python driver (#556)

* PECOBLR-86 Improve logging for debug level

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* PECOBLR-86 Improve logging for debug level

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* fixed format

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* used lazy logging

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* changed debug to error logs

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* used lazy logging

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

---------

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* Update github actions run conditions (#569)

More conditions to run github actions

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* Added classes required for telemetry

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* fixed example

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* changed to doc string

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* removed self.telemetry close line

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* grouped classes

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* formatting

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* fixed doc string

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* fixed doc string

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* added more descriptive comments, put dataclasses in a sub-folder

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* fixed default attributes ordering

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* changed file names

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* added enums to models folder

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* removed telemetry batch size

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

---------

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>
Co-authored-by: Shivam Raj <171748731+shivam2680@users.noreply.github.com>
Co-authored-by: Jothi Prakash <jothi.prakash@databricks.com>
Co-authored-by: Madhav Sainanee <madhav.sainanee@databricks.com>
saishreeeee pushed a commit that referenced this pull request Jun 4, 2025
More conditions to run github actions

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>
saishreeeee added a commit that referenced this pull request Jun 4, 2025
* PECOBLR-86 Improve logging for debug level

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* PECOBLR-86 Improve logging for debug level

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* fixed format

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* used lazy logging

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* changed debug to error logs

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* added classes required for telemetry

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* removed TelemetryHelper

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* [PECOBLR-361] convert column table to arrow if arrow present (#551)

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* Update CODEOWNERS (#562)

new codeowners

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* Enhance Cursor close handling and context manager exception management to prevent server side resource leaks (#554)

* Enhance Cursor close handling and context manager exception management

* tests

* fmt

* Fix Cursor.close() to properly handle CursorAlreadyClosedError

* Remove specific test message from Cursor.close() error handling

* Improve error handling in connection and cursor context managers to ensure proper closure during exceptions, including KeyboardInterrupt. Add tests for nested cursor management and verify operation closure on server-side errors.

* add

* add

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* PECOBLR-86 improve logging on python driver (#556)

* PECOBLR-86 Improve logging for debug level

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* PECOBLR-86 Improve logging for debug level

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* fixed format

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* used lazy logging

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* changed debug to error logs

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* used lazy logging

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

---------

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* Update github actions run conditions (#569)

More conditions to run github actions

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* Added classes required for telemetry

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* fixed example

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* changed to doc string

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* removed self.telemetry close line

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* grouped classes

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* formatting

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* fixed doc string

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* fixed doc string

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* added more descriptive comments, put dataclasses in a sub-folder

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* fixed default attributes ordering

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* changed file names

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* added enums to models folder

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

* removed telemetry batch size

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>

---------

Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>
Co-authored-by: Shivam Raj <171748731+shivam2680@users.noreply.github.com>
Co-authored-by: Jothi Prakash <jothi.prakash@databricks.com>
Co-authored-by: Madhav Sainanee <madhav.sainanee@databricks.com>
Signed-off-by: Sai Shree Pradhan <saishree.pradhan@databricks.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants