{"payload":{"feedbackUrl":"https://github.com/orgs/community/discussions/53140","repo":{"id":131619646,"defaultBranch":"master","name":"dagster","ownerLogin":"dagster-io","currentUserCanPush":false,"isFork":false,"isEmpty":false,"createdAt":"2018-04-30T16:30:04.000Z","ownerAvatar":"https://avatars.githubusercontent.com/u/40032576?v=4","public":true,"private":false,"isOrgOwned":true},"refInfo":{"name":"","listCacheKey":"v0:1719974702.0","currentOid":""},"activityList":{"items":[{"before":"9635e2acfc5a43672d1224047de952dafb42bcd8","after":null,"ref":"refs/heads/bengotow-2024-06/FE-411","pushedAt":"2024-07-03T02:45:02.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"bengotow","name":"Ben Gotow","path":"/bengotow","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1037212?s=80&v=4"}},{"before":"473b545922cf5018f9322eac58573cc39ff9db69","after":"a40853c85e29adc73648fbe13ca9579e34ce4a37","ref":"refs/heads/master","pushedAt":"2024-07-03T02:45:01.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"bengotow","name":"Ben Gotow","path":"/bengotow","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1037212?s=80&v=4"},"commit":{"message":"[ui] Provide an explicit Refresh button to update op partition status FE-411 (#22689)\n\n## Summary & Motivation\r\n\r\nA user reported internal inconsistencies on the op job partitions page\r\nhere:\r\nhttps://linear.app/dagster-labs/issue/FE-411/reporting-an-issue-with-the-backfill-launcher-ui-not-consistently\r\n\r\nThere are a few things going on:\r\n\r\n- This page never updates the overall partition status bar after page\r\nload\r\n- This page DOES update the run+step statuses, but only in the selected\r\nrange\r\n- This page DOES update the backfill status shown at the very bottom of\r\nthe page in the table\r\n\r\nThis means that if a backfill is running, it's easy to get this page\r\ninto a state where it's reporting different information in different\r\nplaces, especially the top bar. This is bad because it doesn't even\r\nrefresh when you click \"Launch backfill\", and causes the UI elements in\r\nthe modal (eg: failed checkbox) to behave incorrectly.\r\n\r\nMy guess is that we originally periodically refreshed the overall\r\npartition status bar and took it out for performance reasons. We just\r\ninvested more into perf + caching in #22629 due to backend load, so I\r\nthink the best solution for now is a tried-and-true explicit refresh\r\nbutton, and triggering a refresh when you open the launchpad.\r\n\r\n\r\n![image](https://github.com/dagster-io/dagster/assets/1037212/480bc6f7-9cc1-4cf7-bba6-efdf66fcbac6)\r\n\r\n\r\n## How I Tested These Changes\r\n\r\nTested this by launching a long-running backfill and playing with the\r\npage extensively while it was running.\r\n\r\nCo-authored-by: bengotow ","shortMessageHtmlLink":"[ui] Provide an explicit Refresh button to update op partition status…"}},{"before":null,"after":"e1d37789209101fa6420a9b93e3e0856ef91be30","ref":"refs/heads/merge-asset-graph-computations","pushedAt":"2024-07-03T02:36:55.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"sryza","name":"Sandy Ryza","path":"/sryza","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/654855?s=80&v=4"},"commit":{"message":"merge asset graph computations","shortMessageHtmlLink":"merge asset graph computations"}},{"before":"4f777ffccfc4011c884d8003fb93830753b81c38","after":"315ae59033d1c78201cde07fc18c4554c864bf6f","ref":"refs/heads/nomorefreeze9safe","pushedAt":"2024-07-03T02:04:39.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"gibsondan","name":null,"path":"/gibsondan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8451211?s=80&v=4"},"commit":{"message":"Remove pendulum from dagster core - non-breaking changes\n\nSummary:\nThis removes the lions share of pendulum from the dagster core codebase - but leaves out the removals that would affect the public API (partitions and schedules).\n\nTest Plan: BK","shortMessageHtmlLink":"Remove pendulum from dagster core - non-breaking changes"}},{"before":"5120be8f24eae11b95c4418ce78ce19af26a5f6d","after":null,"ref":"refs/heads/runrecorddatetime","pushedAt":"2024-07-03T02:04:07.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"gibsondan","name":null,"path":"/gibsondan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8451211?s=80&v=4"}},{"before":"a9c7f532b104b9a6bf6456551b46ff6cc9016593","after":"473b545922cf5018f9322eac58573cc39ff9db69","ref":"refs/heads/master","pushedAt":"2024-07-03T02:04:06.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"gibsondan","name":null,"path":"/gibsondan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8451211?s=80&v=4"},"commit":{"message":"Make run record datetime fields in UTC instead of naive datetimes, remove datetime_as_float (#22818)\n\nSummary:\r\nThis brings run records in OSS in line with run records that are\r\nreturned over our API in that it has non-naive datetimes - UTC datetimes\r\nare strictly better than naive datetimes. It also allows us to remove\r\nthis datetime_as_float utiliyt method which was designed as a way to\r\nreliably handle these naive datetimes coming out of sqlite - my claim is\r\nthat the new syntax that I have replaced it with\r\n(utc_datetime_from_naive(...).timestamp()) is much clearer about what is\r\ngoing on.\r\n\r\nSomewhat unfortunate that the field on RunRecord is \"update_timestamp\"\r\nand not \"update_time\", but that would be a breaking change.\r\n\r\n## Summary & Motivation\r\n\r\n## How I Tested These Changes","shortMessageHtmlLink":"Make run record datetime fields in UTC instead of naive datetimes, re…"}},{"before":"3b55eba1678a1b9db3043e56932c19db35328fda","after":"8845e151f3798287a4925b18ae8f6fa9816a90e7","ref":"refs/heads/nomorefreeze9","pushedAt":"2024-07-03T01:15:00.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"gibsondan","name":null,"path":"/gibsondan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8451211?s=80&v=4"},"commit":{"message":"[9/n] fully remove pendulum from OSS","shortMessageHtmlLink":"[9/n] fully remove pendulum from OSS"}},{"before":"246ed8c8ba5c0181ec8597a987aedc32a3334b15","after":"4f777ffccfc4011c884d8003fb93830753b81c38","ref":"refs/heads/nomorefreeze9safe","pushedAt":"2024-07-03T01:14:39.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"gibsondan","name":null,"path":"/gibsondan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8451211?s=80&v=4"},"commit":{"message":"Remove pendulum from dagster core - non-breaking changes\n\nSummary:\nThis removes the lions share of pendulum from the dagster core codebase - but leaves out the removals that would affect the public API (partitions and schedules).\n\nTest Plan: BK","shortMessageHtmlLink":"Remove pendulum from dagster core - non-breaking changes"}},{"before":"2eea3d63ad72c3d24d2bfac1dfc390199fd2ce2b","after":"246ed8c8ba5c0181ec8597a987aedc32a3334b15","ref":"refs/heads/nomorefreeze9safe","pushedAt":"2024-07-03T01:05:57.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"gibsondan","name":null,"path":"/gibsondan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8451211?s=80&v=4"},"commit":{"message":"Remove pendulum from dagster core - non-breaking changes\n\nSummary:\nThis removes the lions share of pendulum from the dagster core codebase - but leaves out the removals that would affect the public API (partitions and schedules).\n\nTest Plan: BK","shortMessageHtmlLink":"Remove pendulum from dagster core - non-breaking changes"}},{"before":"735862ecaba0c425631b575de0c6fa5a294c7ec0","after":"3b5fe5fdaf18cb0610fce3be9191b37b5db43607","ref":"refs/heads/bengotow-2024-06/FE-406","pushedAt":"2024-07-03T01:04:42.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"bengotow","name":"Ben Gotow","path":"/bengotow","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1037212?s=80&v=4"},"commit":{"message":"Rebase","shortMessageHtmlLink":"Rebase"}},{"before":"53b605c0f1695ba518e619bde826afd75cc7c9ff","after":"5120be8f24eae11b95c4418ce78ce19af26a5f6d","ref":"refs/heads/runrecorddatetime","pushedAt":"2024-07-03T00:55:41.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"gibsondan","name":null,"path":"/gibsondan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8451211?s=80&v=4"},"commit":{"message":"Combine dagster._utils.utc_datetime_from_timestamp with dagster._time.datetime_from_timestamp and move utc_datetime_from_naive into dagster._time\n\nSummary:\nConsolidating all the datetime utils into one place.","shortMessageHtmlLink":"Combine dagster._utils.utc_datetime_from_timestamp with dagster._time…"}},{"before":"896ff57a83dc77ac2a2ed9ccda84ee3136c75894","after":null,"ref":"refs/heads/utilstotime","pushedAt":"2024-07-03T00:55:10.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"gibsondan","name":null,"path":"/gibsondan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8451211?s=80&v=4"}},{"before":"26f4f527269055a1f84274fd540aec48bd0d80c7","after":"a9c7f532b104b9a6bf6456551b46ff6cc9016593","ref":"refs/heads/master","pushedAt":"2024-07-03T00:55:07.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"gibsondan","name":null,"path":"/gibsondan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8451211?s=80&v=4"},"commit":{"message":"Combine dagster._utils.utc_datetime_from_timestamp with dagster._time.datetime_from_timestamp and move utc_datetime_from_naive into dagster._time (#22820)\n\nSummary:\r\nConsolidating all the datetime utils into one place.\r\n\r\n## Summary & Motivation\r\n\r\n## How I Tested These Changes","shortMessageHtmlLink":"Combine dagster._utils.utc_datetime_from_timestamp with dagster._time…"}},{"before":"d0ac4875ec90bc4828b8685f0be738677f6fc0be","after":"735862ecaba0c425631b575de0c6fa5a294c7ec0","ref":"refs/heads/bengotow-2024-06/FE-406","pushedAt":"2024-07-03T00:30:38.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"bengotow","name":"Ben Gotow","path":"/bengotow","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1037212?s=80&v=4"},"commit":{"message":"Rebase","shortMessageHtmlLink":"Rebase"}},{"before":"4ce0f20bddee2d39df142d580e03339303f6d49d","after":"9635e2acfc5a43672d1224047de952dafb42bcd8","ref":"refs/heads/bengotow-2024-06/FE-411","pushedAt":"2024-07-03T00:29:33.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"bengotow","name":"Ben Gotow","path":"/bengotow","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1037212?s=80&v=4"},"commit":{"message":"FE-411: Provide an explicit Refresh button to update op partition status","shortMessageHtmlLink":"FE-411: Provide an explicit Refresh button to update op partition status"}},{"before":"017d4dfa240a1e338e02e3e013e4c39accb2f3fd","after":null,"ref":"refs/heads/bengotow-2024-06/refactor-helper","pushedAt":"2024-07-03T00:28:33.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"bengotow","name":"Ben Gotow","path":"/bengotow","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1037212?s=80&v=4"}},{"before":"a538286c3eab8dbe70b5c223e457b1ca972ccf8d","after":"26f4f527269055a1f84274fd540aec48bd0d80c7","ref":"refs/heads/master","pushedAt":"2024-07-03T00:28:31.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"bengotow","name":"Ben Gotow","path":"/bengotow","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1037212?s=80&v=4"},"commit":{"message":"[ui] Move mocks for viewportClientRect to mocking utils (#22795)\n\n## Summary & Motivation\r\n\r\nJust added a 5th test file that needed this code, figured I'd move it to\r\na shared helper. The shape of the function is a bit different than other\r\nmocks but I think that's required for this particular case.\r\n\r\n## How I Tested These Changes\r\n\r\nTests still pass\r\n\r\nCo-authored-by: bengotow ","shortMessageHtmlLink":"[ui] Move mocks for viewportClientRect to mocking utils (#22795)"}},{"before":"1f60e4da78b594284d14c99620e2a2436ba1ea38","after":null,"ref":"refs/heads/revert-22753-revert-22691-bengotow-2024-06/FE-397","pushedAt":"2024-07-03T00:28:04.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"bengotow","name":"Ben Gotow","path":"/bengotow","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1037212?s=80&v=4"}},{"before":"58ad7b0ffb7640c02a09b77067b3eb9acd9a0436","after":"a538286c3eab8dbe70b5c223e457b1ca972ccf8d","ref":"refs/heads/master","pushedAt":"2024-07-03T00:28:02.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"bengotow","name":"Ben Gotow","path":"/bengotow","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1037212?s=80&v=4"},"commit":{"message":"[ui] Ability to report asset materializations from the actions dropdown [2] (#22781)\n\nThis is a second pass at merging\r\nhttps://github.com/dagster-io/dagster/pull/22691 -- I merged it earlier\r\nthis week without realizing the internal PR\r\n(https://github.com/dagster-io/internal/pull/10341) still had a\r\nbuildkite error on it, and they need to merge together.","shortMessageHtmlLink":"[ui] Ability to report asset materializations from the actions dropdo…"}},{"before":null,"after":"ad8953f271fe2006d68e1d5302923aec17434a56","ref":"refs/heads/jamie/prototype-3","pushedAt":"2024-07-02T22:25:22.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"jamiedemaria","name":null,"path":"/jamiedemaria","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/19783112?s=80&v=4"},"commit":{"message":"v3","shortMessageHtmlLink":"v3"}},{"before":null,"after":"41646ce228034881ca079cf53e5c9db3cc59373e","ref":"refs/heads/al/07-02-_graphql_request_cache_cachingdatatimeresolver","pushedAt":"2024-07-02T22:23:05.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"alangenfeld","name":"Alex Langenfeld","path":"/alangenfeld","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/202219?s=80&v=4"},"commit":{"message":"[graphql] request cache CachingDataTimeResolver","shortMessageHtmlLink":"[graphql] request cache CachingDataTimeResolver"}},{"before":null,"after":"7d78380e1e180a8117a0583958409f8e7cd9608f","ref":"refs/heads/erin/fix-typo-in-code-locs","pushedAt":"2024-07-02T22:14:00.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"erinkcochran87","name":"Erin Cochran","path":"/erinkcochran87","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/16615212?s=80&v=4"},"commit":{"message":"Typo","shortMessageHtmlLink":"Typo"}},{"before":null,"after":"2eea3d63ad72c3d24d2bfac1dfc390199fd2ce2b","ref":"refs/heads/nomorefreeze9safe","pushedAt":"2024-07-02T21:35:57.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"gibsondan","name":null,"path":"/gibsondan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8451211?s=80&v=4"},"commit":{"message":"Remove pendulum from dagster core - non-breaking changes\n\nSummary:\nThis removes the lions share of pendulum from the dagster core codebase - but leaves out the removals that would affect the public API (partitions and schedules).\n\nTest Plan: BK","shortMessageHtmlLink":"Remove pendulum from dagster core - non-breaking changes"}},{"before":"2390416e0eeb0ca9d1534be8792620b84f217908","after":null,"ref":"refs/heads/rl/properly-create-dbt-relation","pushedAt":"2024-07-02T21:34:06.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"rexledesma","name":"Rex Ledesma","path":"/rexledesma","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/16431325?s=80&v=4"}},{"before":"4e0c5fe72af6c3c9025e7b636d7e04a8207672b6","after":"58ad7b0ffb7640c02a09b77067b3eb9acd9a0436","ref":"refs/heads/master","pushedAt":"2024-07-02T21:34:04.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rexledesma","name":"Rex Ledesma","path":"/rexledesma","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/16431325?s=80&v=4"},"commit":{"message":"fix(dbt): use correct relation identifier when creating `BaseRelation` (#22823)\n\n## Summary & Motivation\nWhen creating the `BaseRelation`, we want to use the name of the table\nthat's created in the data warehouse so that we can properly query the\nrelation.\n\nPreviously, we were using `dbt_resource_props[\"name\"]` as the\nidentifier, which is the dbt's reference to that table. Instead, we\nshould use the underlying identifier name.\n\nFor dbt sources, this is `dbt_resource_props[\"identifier\"]`. For dbt\nmodels and seeds, this is `dbt_resource_props[\"alias\"]`.\n\n## How I Tested These Changes\npytest: added dbt aliases for source and model in existing\n`test_dagster_metadata` project","shortMessageHtmlLink":"fix(dbt): use correct relation identifier when creating BaseRelation ("}},{"before":"b690c03c0c62f64c06167d99b5f31a206f4718ad","after":"2390416e0eeb0ca9d1534be8792620b84f217908","ref":"refs/heads/rl/properly-create-dbt-relation","pushedAt":"2024-07-02T21:33:01.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"rexledesma","name":"Rex Ledesma","path":"/rexledesma","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/16431325?s=80&v=4"},"commit":{"message":"fix(dbt): use correct relation identifier when creating `BaseRelation`","shortMessageHtmlLink":"fix(dbt): use correct relation identifier when creating BaseRelation"}},{"before":"00aa8218208066d51370e61a318452887e09dd97","after":null,"ref":"refs/heads/rl/no-quotes-when-retrieving-lineage","pushedAt":"2024-07-02T21:32:38.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"rexledesma","name":"Rex Ledesma","path":"/rexledesma","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/16431325?s=80&v=4"}},{"before":"f1057816b97b7439a9ee62797d6dbd2795c6543d","after":"4e0c5fe72af6c3c9025e7b636d7e04a8207672b6","ref":"refs/heads/master","pushedAt":"2024-07-02T21:32:36.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rexledesma","name":"Rex Ledesma","path":"/rexledesma","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/16431325?s=80&v=4"},"commit":{"message":"fix(dbt): divest from the quoted `relation_name` when building column lineage (#22821)\n\n## Summary & Motivation\n\n- Consolidate the column lineage metadata logic together in a single\nconditional with a try/except similar to\nhttps://github.com/dagster-io/dagster/pull/22641.\n- Divest from using the quoted `relation_name`, and just use the\n`database`, `schema`, `name`.\n- Apply https://github.com/dagster-io/dagster/pull/22642 when building\nthe column lineage.\n\n## How I Tested These Changes\nThis should probably be under nightly test against a real bigquery\ninstance.\n\nLocally, things work now.\n\n### Before\n\nMaterializing with `.fetch_column_metadata()` resulted in:\n\n```\ndagster._core.errors.DagsterExecutionStepExecutionError: Error occurred while executing op \"jaffle_shop_dbt_assets\":\nThe above exception was caused by the following exception:\ngoogle.api_core.exceptions.BadRequest: 400 GET https://bigquery.googleapis.com/bigquery/v2/projects/%60dagster-cloud-local-test%60/datasets/%60toys%60/tables/%60raw_customers%60?prettyPrint=false: Invalid resource name projects/`dagster-cloud-local-test`; Project id: `dagster-cloud-local-test`\n```\n\n\n### After\n\nBigquery assets materialize successfully with metadata:\n\n![image](https://github.com/dagster-io/dagster/assets/16431325/63867fcf-d404-44ca-a06c-60989c79b744)","shortMessageHtmlLink":"fix(dbt): divest from the quoted relation_name when building column…"}},{"before":"867a6118be54f03ea501488676f3b0fa6b7da5ff","after":"3b55eba1678a1b9db3043e56932c19db35328fda","ref":"refs/heads/nomorefreeze9","pushedAt":"2024-07-02T21:24:00.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"gibsondan","name":null,"path":"/gibsondan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8451211?s=80&v=4"},"commit":{"message":"[9/n] fully remove pendulum from OSS","shortMessageHtmlLink":"[9/n] fully remove pendulum from OSS"}},{"before":"87852384615bc1ac4b51e0f6a993cfd5da831929","after":"53b605c0f1695ba518e619bde826afd75cc7c9ff","ref":"refs/heads/runrecorddatetime","pushedAt":"2024-07-02T21:05:40.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"gibsondan","name":null,"path":"/gibsondan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8451211?s=80&v=4"},"commit":{"message":"Combine dagster._utils.utc_datetime_from_timestamp with dagster._time.datetime_from_timestamp and move utc_datetime_from_naive into dagster._time\n\nSummary:\nConsolidating all the datetime utils into one place.","shortMessageHtmlLink":"Combine dagster._utils.utc_datetime_from_timestamp with dagster._time…"}}],"hasNextPage":true,"hasPreviousPage":false,"activityType":"all","actor":null,"timePeriod":"all","sort":"DESC","perPage":30,"cursor":"djE6ks8AAAAEdWk_mAA","startCursor":null,"endCursor":null}},"title":"Activity · dagster-io/dagster"}