Re: [PR] Speed up autocompletion of Breeze by simplifying provider state [airflow]

2023-12-30 Thread via GitHub
potiuk merged PR #36499: URL: https://github.com/apache/airflow/pull/36499 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@airflow.a

Re: [PR] Speed up autocompletion of Breeze by simplifying provider state [airflow]

2023-12-30 Thread via GitHub
potiuk commented on PR #36499: URL: https://github.com/apache/airflow/pull/36499#issuecomment-1872577644 Merging. the Mssql failures already fixed in main -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above t

Re: [PR] Speed up autocompletion of Breeze by simplifying provider state [airflow]

2023-12-30 Thread via GitHub
potiuk commented on PR #36499: URL: https://github.com/apache/airflow/pull/36499#issuecomment-1872571541 BTW. py-spy is great :) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comme

Re: [PR] Speed up autocompletion of Breeze by simplifying provider state [airflow]

2023-12-30 Thread via GitHub
potiuk commented on PR #36499: URL: https://github.com/apache/airflow/pull/36499#issuecomment-1872570351 This had been discussed in https://github.com/apache/airflow/pull/36391 and it turned out that complex state management had real impact on auto-complete: The auto-completion time w

Re: [PR] Speed up autocompletion of Breeze by simplifying provider state [airflow]

2023-12-30 Thread via GitHub
potiuk closed pull request #36498: Speed up autocompletion of Breeze by simplifying provider state URL: https://github.com/apache/airflow/pull/36498 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to t

Re: [PR] Speed up autocompletion of Breeze by simplifying provider state [airflow]

2023-12-30 Thread via GitHub
potiuk commented on PR #36498: URL: https://github.com/apache/airflow/pull/36498#issuecomment-1872570320 Closing in favour of #36499 - because this change should be done in "apache" repo -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[PR] Speed up autocompletion of Breeze by simplifying provider state [airflow]

2023-12-30 Thread via GitHub
potiuk opened a new pull request, #36499: URL: https://github.com/apache/airflow/pull/36499 Some recent changes, adding removed and suspended state for breeze caused significant slow-down of autocompletion retrieval - as it turned out, because we loaded and parsed all provider yaml files du

Re: [PR] Speed up autocompletion of Breeze by simplifying provider state [airflow]

2023-12-30 Thread via GitHub
potiuk commented on PR #36498: URL: https://github.com/apache/airflow/pull/36498#issuecomment-1872569691 This had been discussed in https://github.com/apache/airflow/pull/36391 and it turned out that complex state management had real impact on auto-complete: The auto-completion time w

[PR] Speed up autocompletion of Breeze by simplifying provider state [airflow]

2023-12-30 Thread via GitHub
potiuk opened a new pull request, #36498: URL: https://github.com/apache/airflow/pull/36498 Some recent changes, adding removed and suspended state for breeze caused significant slow-down of autocompletion retrieval - as it turned out, because we loaded and parsed all provider yaml files du