Parag Jain
Parag Jain
@jihoonson The proposal looks great, I did this PR because bytes ingested is not available for streaming or batch tasks. I see your proposal only includes metrics for batch tasks,...
> To do so, I have been thinking to add new classes each of which defines all metrics useful for batch and streaming ingestion, respectively. I'm not sure why we...
@jihoonson are you actively working on your proposal ? do you think you can reuse the `InputStats` strategy from here ?
As long as we get the metrics about how many raw bytes are processed from the source (including scans for determining shard specs) I think I am ok with any...
Makes sense, so as long as https://github.com/apache/druid/pull/10407#issuecomment-713016099 is satisfied things seems good to me.
hey @somu-imply, I can look into resolving conflicts over the weekend. However, @jihoonson was working on something similar iirc so not sure whats the status of it and if this...
Don't see documentation changes in this PR like this - https://github.com/apache/druid/pull/10407/files#diff-4153c0d20e748deebfabfa5f90696f686d98ff83362e7fea73937a025ed42db5
@crirus we are running `0.9.42` version of swiv against latest druid stable (`0.10.0`). Have not seen any issues so far.
Apart from what @cheddar suggested here are some debugging tips - try running swiv with verbose (-v) option and see if you find something meaningful, check your config (has it...
- Add more test cases - Support for protobuf serialization