Nora Howard

I think pycodestyle's handling of binary operator expressions that are the values of kwargs could be improved. There is a case that is a little ambiguous to me as to...

Use input paths as keys when producing json from metacp

1

If I pass relative paths to metacp, it converts them to absolute paths in the output json object's keys. I think it would be better if metacp used the same...

Shading code with lambdas fails at runtime

1

I ran into this when trying to shade some code I'd written with lambdas. I think this is the jarjar equivalent of https://issues.apache.org/jira/browse/MSHADE-260 I could work up a proof of...

'AssertionError: Failed to cache distribution' is difficult to diagnose

2

From reading the code around the assertion, I think there's a number of ways that the cache distribution might be missing something. I think it'd be nice if the errors...

OpenSearchAdapter - Include field name in error log about map update failure

2

### Summary When the OpenSearchAdapter has an error doing a map update for a field, it doesn't include the field name in the log message. This adds the field name...

### Summary Instead of adding the grpc.service metric decorator to all services declared with the builder, scope it to the grpc service. When I was looking at the decorator usage...

[BUG] when snapshot metadata has no size, it's treated as live even if it is not

1

### Describe the bug Trying to upgrade an index node that is based on an image from before https://github.com/slackhq/astra/pull/940 to one after https://github.com/slackhq/astra/pull/1030 will cause it to attempt to delete...

### Describe the bug https://github.com/slackhq/astra/pull/968 made transactions optional. But there is code underneath it that assumes it happens in a transaction and discards a future from the producer client: https://github.com/slackhq/astra/blob/master/astra/src/main/java/com/slack/astra/bulkIngestApi/BulkIngestKafkaProducer.java#L336-L339...