mengw15 commented on PR #6016: URL: https://github.com/apache/texera/pull/6016#issuecomment-5030823889
Two mechanical notes on the current `DatasetResourceSpec`: scalafmt is failing on it (and it's missing a trailing newline), so CI exits before running any test. And although it's (correctly) meant to be a unit test — untagged, so it runs on the `amber` unit leg — it drives live LakeFS/S3 through the full `searchAllResourcesCall` path, which the unit leg doesn't provision, so it'll fail reaching LakeFS once scalafmt is cleared. The query-level spec above keeps it a real unit test by not going through `toEntryImpl` at all. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
