KR-bluejay comments

Results 14 comments of


                                            KR-bluejay

job data cleanup does not work if `pull-staged` strategy selected

Hi @milenkovicm , I would like to take this issue. In PullStaged, the scheduler can’t call executors directly to clean job data. Proposal: extend PollWorkResult with CleanJobDataParams so that executors...

job data cleanup does not work if `pull-staged` strategy selected

Thanks! I'll take it.

job data cleanup does not work if `pull-staged` strategy selected

Hi, @milenkovicm I have implemented the pull-based cleanup, but I’m not sure about two things: (Currently, the PR is still in draft mode. #1314) 1. **Tests** The scheduler keeps the...

job data cleanup does not work if `pull-staged` strategy selected

Got it, thank you for the update! I'll wait for your feedback.

fix: ensure transaction rollback for duplicate ID inserts

This is my first PR to SurrealDB, and I'm quite nervous about it! 😅 I've tried my best to follow the contributing guidelines and PR template, but as this is...

fix: ensure transaction rollback for duplicate ID inserts

@emmanuel-keller Thank you for reviewing my changes! I've made the following updates based on your feedback: 1. Modified `insert.rs` to ensure proper transaction rollback: - Previously, when `retryable` was true,...

Expose Logical and Physical plan details in the REST API

Hi, I’d be happy to contribute to this. Would it be okay if I take this issue on? Also, could you share a bit more detail about what you’d like...

Expose Logical and Physical plan details in the REST API

Okay, I’ll look into it.

[EPIC] Improve job data cleanup logic

I’d be happy to work on this as a follow-up, if this direction sounds good.

[EPIC] Improve job data cleanup logic

Thanks for the great ideas. As I'm still getting familiar with the internals of Ballista, I might be missing some key points, but I wanted to share a few thoughts...