Bashir Sadjad
Bashir Sadjad
Setting the priority to P3 until AMPATH resumes their evaluation work.
This is not high priority for now (hence P3) because conversion to HAPI objects and then JSON encoding, adds the `resourceType` field.
Thanks @meetmehta1198 for this and congratulations on your first contribution to this repo. Just one minor comment: It seems to me that from #1103 you are fixing tests under the...
@chandrashekar-s I pushed [another commit](https://github.com/google/fhir-data-pipes/pull/1342/commits/cfd867443565078a91cc9dd5b297f23072e3cad0) which covers more edge cases for view recreation. Please let me know when you start reviewing this PR such that I don't update it while...
@chandrashekar-s I am going to merge this since it may be needed for someone else's experiments. Please finish your review whenever you can and share your comments; I'll apply them...
@chandrashekar-s I think this is ready for review (I'll remove the Maven log file from this PR later; keeping it for now to have a sample of new WARNING messages)....
> Apologies for the delayed review and thanks @bashir2 for initiating this change, with this change lot of potential NPE can be avoided. > > I had high level look...
Can you please provide a sample config (or command line if you are running the pipelines directly and not through the `controller`) to reproduce this? Also please provide the full...
Actually this feature is the long standing issue #455, i.e., adding BigQuery as a sink option. It should not be too hard to add this and I think it is...