Thanks patch applied. On Thu, May 30, 2019 at 5:01 PM Murtuza Zabuawala < murtuza.zabuaw...@enterprisedb.com> wrote:
> Here is updated patch. > > On Thu, May 30, 2019 at 4:06 PM Aditya Toshniwal < > aditya.toshni...@enterprisedb.com> wrote: > >> Hi Murtuza, >> >> I was just going through code change, and I would suggest one small >> change. >> In the image_mapper.js, export it as default, as it will have only one >> export. Then you can use it as imageMapper.default rather than >> imageMapper.imageMapper. >> Otherwise, looks fine to me. I didn't test though :P >> >> >> On Thu, May 30, 2019 at 2:41 PM Murtuza Zabuawala < >> murtuza.zabuaw...@enterprisedb.com> wrote: >> >>> Hi, >>> >>> Embedding images using *.toDataURL()* method hits the performance of >>> the explain plan rendering if the plan is complex because first it >>> downloads the images then it will convert each images into base64, to >>> improve the performance we will embed the images only when downloading of >>> SVG is called and not when displaying the graphical plan. >>> >>> -- >>> Regards, >>> Murtuza Zabuawala >>> EnterpriseDB: http://www.enterprisedb.com >>> The Enterprise PostgreSQL Company >>> >>> >> >> -- >> Thanks and Regards, >> Aditya Toshniwal >> Software Engineer | EnterpriseDB India | Pune >> "Don't Complain about Heat, Plant a TREE" >> > -- *Thanks & Regards* *Akshay Joshi* *Sr. Software Architect* *EnterpriseDB Software India Private Limited* *Mobile: +91 976-788-8246*
