Thread: Patch: update BigSQL links to https

Patch: update BigSQL links to https

From
"Rader, David"
Date:
Attached is a patch to update the download links for BigSQL to https, and update the name on the Linux download page from OpenSCG to BigSQL. Can you please review for commit?

Thank you
Dave

--
David Rader
Attachment

Re: Patch: update BigSQL links to https

From
"Jonathan S. Katz"
Date:
Hi,

> On Sep 7, 2016, at 3:57 PM, Rader, David <davidr@openscg.com> wrote:
>
> Attached is a patch to update the download links for BigSQL to https, and update the name on the Linux download page
fromOpenSCG to BigSQL. Can you please review for commit? 

Mosty looks ok.  There was some extra whitespace added in around this line
+and integration with external databases (Cassandra, Oracle, SQL Server, Hadoop),

Please clean up and I can apply it.

Thanks!

Jonathan


Re: Patch: update BigSQL links to https

From
"Rader, David"
Date:
On Thu, Sep 8, 2016 at 11:35 AM, Jonathan S. Katz <jkatz@postgresql.org> wrote:
Hi,

> On Sep 7, 2016, at 3:57 PM, Rader, David <davidr@openscg.com> wrote:
>
> Attached is a patch to update the download links for BigSQL to https, and update the name on the Linux download page from OpenSCG to BigSQL. Can you please review for commit?

Mosty looks ok.  There was some extra whitespace added in around this line

        +and integration with external databases (Cassandra, Oracle, SQL Server, Hadoop),

Please clean up and I can apply it.

Thanks!

Jonathan

Here is an updated version of the patch with the extra whitespace removed.

Thanks
Dave

Attachment

Re: Patch: update BigSQL links to https

From
"Jonathan S. Katz"
Date:

On Sep 8, 2016, at 11:59 AM, Rader, David <davidr@openscg.com> wrote:

On Thu, Sep 8, 2016 at 11:35 AM, Jonathan S. Katz <jkatz@postgresql.org> wrote:
Hi,

> On Sep 7, 2016, at 3:57 PM, Rader, David <davidr@openscg.com> wrote:
>
> Attached is a patch to update the download links for BigSQL to https, and update the name on the Linux download page from OpenSCG to BigSQL. Can you please review for commit?

Mosty looks ok.  There was some extra whitespace added in around this line

        +and integration with external databases (Cassandra, Oracle, SQL Server, Hadoop),

Please clean up and I can apply it.

Thanks!

Jonathan

Here is an updated version of the patch with the extra whitespace removed.

Thanks.  Applied.

Jonathan