Bashir Sadjad

@mozzy11 and @muhammad-levi, I thought it is best to continue the discussion re. [this comment](https://github.com/google/fhir-data-pipes/pull/947#discussion_r1489848068) here in the issue (instead of the PR) for better visibility. Can you both please...

Thanks @machariamuguku for filing this. Is the main motivation for this to tune performance? Did you do any experiments with other numbers (I mean by running the pipeline directly) and...

PR #813 added some basic support for running Sonar on our code but it is not integrated in our CI yet.

An update on this issue: In PR #1026 I tried to address this issue by adding a `fullId` option to keep the full ID when creating Avro records. This turned...

As noted in the last comment above, including resource-type or other information (e.g., url) in `id` field of Avro records, is wrong. The original issue reported in OP actually goes...

@jjtswan can we either consolidate between this and #609 or make it very clear that this issue is for extending our test-suite and the other is pure documentation? With current...

@williamito can you please take a look at this and either update documentation or close this if with your changes we already have this covered?

I am closing this as our new documentation site reflects this to some extent; especially once PR #1110 is merged.

For reference: We have done #266 before and its findings are well documented [here](https://github.com/google/fhir-data-pipes/issues/266#issuecomment-1212505139). A big part of the performance question for the pipelines is about the throughput of the...

> Merger pipeline - you mean for the incremental updates for micro-batch? Yes for the incremental updates and to be more specific, I mean the [merger pipeline](https://github.com/google/fhir-data-pipes/blob/master/pipelines/batch/src/main/java/org/openmrs/analytics/ParquetMerger.java) as it reads...