Thanks patch applied.

On Thu, Aug 29, 2019 at 11:18 AM Aditya Toshniwal <
aditya.toshni...@enterprisedb.com> wrote:

>
> On Thu, Aug 29, 2019 at 10:59 AM Ashesh Vashi <
> ashesh.va...@enterprisedb.com> wrote:
>
>> On Wed, Aug 28, 2019 at 1:32 PM Aditya Toshniwal <
>> aditya.toshni...@enterprisedb.com> wrote:
>>
>>> Hi Hackers,
>>>
>>> Attached is the patch to increase cache control max age (in http header)
>>> for static files to improve performance over longer run.
>>> By default it is 43200 seconds (12 hrs), which means that the static
>>> files like vendor.js, etc. will be fetched again from server after 12hrs.
>>> This is not required as these files won't change.
>>>
>>> Kindly review.
>>>
>> I have done the review, it looks good to me.
>> I have only concern about - with this change the caching will be done for
>> at least 1 year. I am still kind of confused, what should be the caching
>> time?
>>
>> But - we can go ahead with this change.
>>
> Thank you Ashesh. I had set it to a year as the user may not upgrade
> pgAdmin with every release. Plus, we have cache busting mechanism (version
> appended to URL). Nevertheless, we can may be set it to 30 days ( 2592000
> seconds), which is the release cycle interval for pgAdmin.
>
>>
>> --
>>
>> Thanks & Regards,
>>
>> Ashesh Vashi
>> EnterpriseDB INDIA: Enterprise PostgreSQL Company
>> <http://www.enterprisedb.com/>
>>
>>
>> *http://www.linkedin.com/in/asheshvashi*
>> <http://www.linkedin.com/in/asheshvashi>
>>
>>
>>
>>>
>>> --
>>> Thanks and Regards,
>>> Aditya Toshniwal
>>> Software Engineer | EnterpriseDB India | Pune
>>> "Don't Complain about Heat, Plant a TREE"
>>>
>>
>
> --
> 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*

Reply via email to