spelling of enable_partition_wise_join
I wonder how others feel about this, but the spelling of
enable_partition_wise_join feels funny to me every time I look at it. I
would write it enable_partitionwise_join.
Thoughts?
--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
Peter Eisentraut wrote:
I wonder how others feel about this, but the spelling of
enable_partition_wise_join feels funny to me every time I look at it. I
would write it enable_partitionwise_join.
+1
See also: /messages/by-id/20171005134847.shzldz2ublrb3ny2@alvherre.pgsql
--
�lvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
On 14/02/18 09:27, Peter Eisentraut wrote:
I wonder how others feel about this, but the spelling of
enable_partition_wise_join feels funny to me every time I look at it. I
would write it enable_partitionwise_join.Thoughts?
As 'wise' is a suffix, not a word in this situation - so it be combined
with 'partition'.
Hence 'enable_partitionwise_join' is more correct.
Cheers,
Gavin
On Wed, Feb 14, 2018 at 2:15 AM, Alvaro Herrera <alvherre@alvh.no-ip.org> wrote:
Peter Eisentraut wrote:
I wonder how others feel about this, but the spelling of
enable_partition_wise_join feels funny to me every time I look at it. I
would write it enable_partitionwise_join.+1
See also: /messages/by-id/20171005134847.shzldz2ublrb3ny2@alvherre.pgsql
To that I replied with
/messages/by-id/CAFjFpRcsZnxCen88a-16R5EYqPCwFYnFThM+mjagu=B1QvxPVA@mail.gmail.com.
I didn't get any further response, so nothing was changed that time.
Alvaro, Peter, Gavin have voted for partitionwise in this thread and
Robert had similar objections earlier. Looks like we should change it.
--
Best Wishes,
Ashutosh Bapat
EnterpriseDB Corporation
The Postgres Database Company
On 2/14/18 03:28, Ashutosh Bapat wrote:
On Wed, Feb 14, 2018 at 2:15 AM, Alvaro Herrera <alvherre@alvh.no-ip.org> wrote:
Peter Eisentraut wrote:
I wonder how others feel about this, but the spelling of
enable_partition_wise_join feels funny to me every time I look at it. I
would write it enable_partitionwise_join.+1
See also: /messages/by-id/20171005134847.shzldz2ublrb3ny2@alvherre.pgsql
To that I replied with
/messages/by-id/CAFjFpRcsZnxCen88a-16R5EYqPCwFYnFThM+mjagu=B1QvxPVA@mail.gmail.com.
I didn't get any further response, so nothing was changed that time.
Alvaro, Peter, Gavin have voted for partitionwise in this thread and
Robert had similar objections earlier. Looks like we should change it.
done
--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
On Fri, Feb 16, 2018 at 9:09 PM, Peter Eisentraut
<peter.eisentraut@2ndquadrant.com> wrote:
On 2/14/18 03:28, Ashutosh Bapat wrote:
On Wed, Feb 14, 2018 at 2:15 AM, Alvaro Herrera <alvherre@alvh.no-ip.org> wrote:
Peter Eisentraut wrote:
I wonder how others feel about this, but the spelling of
enable_partition_wise_join feels funny to me every time I look at it. I
would write it enable_partitionwise_join.+1
See also: /messages/by-id/20171005134847.shzldz2ublrb3ny2@alvherre.pgsql
To that I replied with
/messages/by-id/CAFjFpRcsZnxCen88a-16R5EYqPCwFYnFThM+mjagu=B1QvxPVA@mail.gmail.com.
I didn't get any further response, so nothing was changed that time.
Alvaro, Peter, Gavin have voted for partitionwise in this thread and
Robert had similar objections earlier. Looks like we should change it.done
Thanks. There are functions like try_partition_wise_join(),
generate_partition_wise_join_paths() which use partition_wise
spelling. Should we update those as well?
--
Best Wishes,
Ashutosh Bapat
EnterpriseDB Corporation
The Postgres Database Company
On Mon, Feb 19, 2018 at 9:08 AM, Ashutosh Bapat
<ashutosh.bapat@enterprisedb.com> wrote:
Thanks. There are functions like try_partition_wise_join(),
generate_partition_wise_join_paths() which use partition_wise
spelling. Should we update those as well?
I see that the commit already did that. Sorry. Although, I would have
liked comments to spell "partition-wise" instead of "partitionwise".
https://dictionary.cambridge.org/dictionary/english/wise
differentiates between "-wise" and "wise". The later is used when
indicating a direction and the first one indicates "relating to". I
think here we want the first one.
--
Best Wishes,
Ashutosh Bapat
EnterpriseDB Corporation
The Postgres Database Company