Shivji Kumar Jha

Results 10 comments of Shivji Kumar Jha

Hi @matthiashanel, I work with @goku321 . > @goku321 What's your expectation for an R2 consumer in an R5 stream where both consumer nodes are shut down? > We use...

> @shiv4289 , but loosing the offset is basically what it'd mean if we where to automatically migrate the R1 consumer, once it's machine disappears. > > > Expected result:...

@matthiashanel @derekcollison Hi, we have commented out the consumer migration code on stop server and tested the same. We are using this modified server binary and it works in a...

Hi @matthiashanel any progress on this? Open to collaborate on code/tests if that helps speed this up. Love to have this.

This is an important change for us. Pulsar manager not integrating with our SSO is a huge roadblock for pulsar manager adoption. Great work @sourabhaggrawal . Lets sprint to merge...

@tuteng @eolivelli Can you please approve the test workflow run?

@vdimir can you help remove the `**submodule changed**` tag, if possible. It comes from a bad commit. I have reverted that part.

The [test changed in this PR](https://github.com/ClickHouse/ClickHouse/pull/81133/files#diff-cf28d8f469541df4d4fa1cc7763e09f0c8edd950909205bef0e7acc444a5516f) is flaky. Investigating why `ProfileEvents['GranulesSkippedByPrewhereReaders']` throws up different results (1021, [1045](https://s3.amazonaws.com/clickhouse-test-reports/json.html?PR=81133&sha=053fed49e25d351ad4bc97d9865318828599500e&name_0=PR&name_1=Stateless%20tests%20%28tsan%2C%20s3%20storage%2C%203%2F3%29), [1014](https://s3.amazonaws.com/clickhouse-test-reports/json.html?PR=81133&sha=053fed49e25d351ad4bc97d9865318828599500e&name_0=PR&name_1=Stateless%20tests%20%28release%2C%20old%20analyzer%2C%20s3%2C%20DatabaseReplicated%2C%202%2F2%29)) on different CI runs. `ProfileEvents['RowsSkippedByPrewhereReaders']` changes too ([9998779 vs 9983539](https://s3.amazonaws.com/clickhouse-test-reports/json.html?PR=81133&sha=053fed49e25d351ad4bc97d9865318828599500e&name_0=PR&name_1=Stateless%20tests%20%28release%2C%20old%20analyzer%2C%20s3%2C%20DatabaseReplicated%2C%202%2F2%29)) Not sure...

I tried to change the test to use: 1. 1 thread ( --max_threads 1 --max_insert_threads 1) 2. remove granule adaptability (index_granularity_bytes=0) Still same failures and flaky across different CI runs!...

Hi @vdimir Another observation from grepping the failure logs (as suggested in slack): The difference seems to be that the lines tagged **DiskLocal** are missing. And thats because, all the...