Zyiqin-Miranda

Results 12 comments of Zyiqin-Miranda

> > # ⚠️ **Performance Alert** ⚠️ > > Possible performance regression was detected for benchmark. Benchmark result of this commit is worse than the previous benchmark result exceeding threshold...

Hi @achsvg, could you also share your Ray cluster YAML config file?

Hi @achsvg, added my comment in this [PR](https://github.com/ray-project/ray/pull/46537#issuecomment-2354671762)

Hi @achsvg, could you share a repro script for this issue? The function of replacing the `instance_id` for alarm is meant to be called by head node only to determine...

Verified that the alarms instance ids are correctly replaced and alarms can be successfully setup for Ray clusters. Attached is the CloudWatch alarms with Instance id specified. ![Screenshot 2024-09-16 at...

Hi @achsvg, just tested and looks like the original code is working, not sure why it's not working in your case, would be great if you could share your Ray...

> > I _think_ `compute` should be the right module for it, since I expect the `compute` module to hold all DeltaCAT "stored-procedure-like-jobs" that we expect to periodically invoke w/...

Thanks @kevinjqliu, I can work on this issue

Thanks @kevinjqliu and @sungwy. Starting to add support to current `plan_files` function for equality deletes, not sure if the current [_InclusiveMetricsEvaluator](https://github.com/apache/iceberg-python/blob/c30e43adf94a82ec1a225d3a1bf69fface592cfd/pyiceberg/expressions/visitors.py#L1137) can be directly used to determine whether the equality...