UPDATED Patch for adding DATACUBE operator

Started by sumitabout 23 years ago8 messagespatches
Jump to latest
#1sumit
sumit@gdit.iiit.net

As per the suggestions received, the syntax for the DATACUBE has
been modified to

SELECT <field list><aggregate list>
INTO <destination table>
FROM <table expression>
WHERE <search condition>
GROUP BY CUBE <aggregate list>
HAVING <search condition>;

Kindly note that the file datacube.c should be placed in src/backend/tcop/
and datacube.h should be in src/include.

Please let us know of further modifications that may be necessary.

Srikanth M
Sumit Kumar Mukherjee

Attachments:

datacube_patchtext/plain; CHARSET=US-ASCII; NAME=datacube_patchDownload+111-14
README.datacubetext/plain; CHARSET=ISO-8859-1; NAME=README.datacubeDownload
datacube.ctext/plain; CHARSET=US-ASCII; NAME=datacube.cDownload
datacube.htext/plain; CHARSET=US-ASCII; NAME=datacube.hDownload
#2sumit
sumit@gdit.iiit.net
In reply to: sumit (#1)
Re: UPDATED Patch for adding DATACUBE operator

Can someone inform us of the developments regarding our code on
DATACUBE? We would like to know if there are somethings which we might
have overlooked. If possible, can we also know if we can expect it to be
added to this release?

Sumit
Srikanth

#3Bruce Momjian
bruce@momjian.us
In reply to: sumit (#1)
Re: UPDATED Patch for adding DATACUBE operator

Your patch has been added to the PostgreSQL unapplied patches list at:

http://momjian.postgresql.org/cgi-bin/pgpatches

I will try to apply it within the next 48 hours.

---------------------------------------------------------------------------

sumit wrote:

As per the suggestions received, the syntax for the DATACUBE has
been modified to

SELECT <field list><aggregate list>
INTO <destination table>
FROM <table expression>
WHERE <search condition>
GROUP BY CUBE <aggregate list>
HAVING <search condition>;

Kindly note that the file datacube.c should be placed in src/backend/tcop/
and datacube.h should be in src/include.

Please let us know of further modifications that may be necessary.

Srikanth M
Sumit Kumar Mukherjee

Content-Description:

[ Attachment, skipping... ]

Content-Description:

[ Attachment, skipping... ]

Content-Description:

[ Attachment, skipping... ]

Content-Description:

[ Attachment, skipping... ]

---------------------------(end of broadcast)---------------------------
TIP 2: you can get off all lists at once with the unregister command
(send "unregister YourEmailAddressHere" to majordomo@postgresql.org)

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073
#4Bruce Momjian
bruce@momjian.us
In reply to: sumit (#1)
Re: UPDATED Patch for adding DATACUBE operator

This looks good, but it is going to require more work, and we have run
out of time for 7.4.

This has been saved for the 7.5 release:

http:/momjian.postgresql.org/cgi-bin/pgpatches2

Sorry.

---------------------------------------------------------------------------

sumit wrote:

As per the suggestions received, the syntax for the DATACUBE has
been modified to

SELECT <field list><aggregate list>
INTO <destination table>
FROM <table expression>
WHERE <search condition>
GROUP BY CUBE <aggregate list>
HAVING <search condition>;

Kindly note that the file datacube.c should be placed in src/backend/tcop/
and datacube.h should be in src/include.

Please let us know of further modifications that may be necessary.

Srikanth M
Sumit Kumar Mukherjee

Content-Description:

[ Attachment, skipping... ]

Content-Description:

[ Attachment, skipping... ]

Content-Description:

[ Attachment, skipping... ]

Content-Description:

[ Attachment, skipping... ]

---------------------------(end of broadcast)---------------------------
TIP 2: you can get off all lists at once with the unregister command
(send "unregister YourEmailAddressHere" to majordomo@postgresql.org)

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073
#5Bruce Momjian
bruce@momjian.us
In reply to: Bruce Momjian (#4)
Re: UPDATED Patch for adding DATACUBE operator

Tom, what changes need to be made, or are you too busy?

---------------------------------------------------------------------------

Srikanth M wrote:

Dear Sir,

Please tell the changes you want to incorporate in the present
DATACUBE operator.

Thanking you

Srikanth
Sumit

On Fri, 25
Jul 2003, Bruce Momjian wrote:

This looks good, but it is going to require more work, and we have run
out of time for 7.4.

This has been saved for the 7.5 release:

http:/momjian.postgresql.org/cgi-bin/pgpatches2

Sorry.

---------------------------------------------------------------------------

sumit wrote:

As per the suggestions received, the syntax for the DATACUBE has
been modified to

SELECT <field list><aggregate list>
INTO <destination table>
FROM <table expression>
WHERE <search condition>
GROUP BY CUBE <aggregate list>
HAVING <search condition>;

Kindly note that the file datacube.c should be placed in src/backend/tcop/
and datacube.h should be in src/include.

Please let us know of further modifications that may be necessary.

Srikanth M
Sumit Kumar Mukherjee

Content-Description:

[ Attachment, skipping... ]

Content-Description:

[ Attachment, skipping... ]

Content-Description:

[ Attachment, skipping... ]

Content-Description:

[ Attachment, skipping... ]

---------------------------(end of broadcast)---------------------------
TIP 2: you can get off all lists at once with the unregister command
(send "unregister YourEmailAddressHere" to majordomo@postgresql.org)

--
If the automobile had followed the same development cycle as the computer, a Rolls-Royce today would cost $100, get a million miles to the gallon, and explode once every few weeks, killing everyone inside.

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073
#6Tom Lane
tgl@sss.pgh.pa.us
In reply to: Bruce Momjian (#5)
Re: UPDATED Patch for adding DATACUBE operator

Bruce Momjian <pgman@candle.pha.pa.us> writes:

Tom, what changes need to be made, or are you too busy?

It needs a ground-up rewrite, and I'm too busy to think about that
anytime soon. I sent a long list of problems as commentary on the
patch about a month ago. Unfortunately I didn't keep a copy, and
I'm not having any luck finding that message in the archives right
now. Sumit, did you keep a copy?

regards, tom lane

#7Kris Jurka
books@ejurka.com
In reply to: Tom Lane (#6)
Re: UPDATED Patch for adding DATACUBE operator

On Fri, 25 Jul 2003, Tom Lane wrote:

Bruce Momjian <pgman@candle.pha.pa.us> writes:

Tom, what changes need to be made, or are you too busy?

It needs a ground-up rewrite, and I'm too busy to think about that
anytime soon. I sent a long list of problems as commentary on the
patch about a month ago. Unfortunately I didn't keep a copy, and
I'm not having any luck finding that message in the archives right
now. Sumit, did you keep a copy?

http://groups.google.com/groups?hl=en&amp;lr=&amp;ie=UTF-8&amp;selm=28882.1057162309%40sss.pgh.pa.us

Kris Jurka

#8Tom Lane
tgl@sss.pgh.pa.us
In reply to: Kris Jurka (#7)
Re: UPDATED Patch for adding DATACUBE operator

Kris Jurka <books@ejurka.com> writes:

On Fri, 25 Jul 2003, Tom Lane wrote:

I'm not having any luck finding that message in the archives right
now. Sumit, did you keep a copy?

http://groups.google.com/groups?hl=en&amp;lr=&amp;ie=UTF-8&amp;selm=28882.1057162309%40sss.pgh.pa.us

Thanks, but that actually wasn't the message I had in mind --- IIRC,
Sumit replied to that one, and I replied with a more detailed set of
complaints. Perhaps this was off-list.

regards, tom lane