br1ghtyang / asterixdb

Automatically exported from code.google.com/p/asterixdb
0 stars 0 forks source link

remove write statement #542

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
And this feature doesn't exist according to the docs, so it can be 
safely/quietly removed!
Cheers,
Mike

On 6/23/13 9:48 PM, Vinayak Borkar wrote:
I think this was implemented a long time ago when we did not have inserts. This 
was a way for us to get a result of a query into a dataset. In the past, we 
always sorted data for a bulk load and so this would have been right. At some 
point we stopped doing that - could that be when this broke?

Anyway, we can get rid of this feature now that we have inserts...

Vinayak

On 6/23/13 9:37 PM, Sattam Alsubaiee wrote:
Does anyone know why do we have the statement "write into dataset"?

WriteStatement ::= "write" ( ( "output" "to" Identifier ":" StringLiteral (
"using" StringLiteral )? ) | ( "into" <DATASET> <LEFTPAREN> Query
<RIGHTPAREN> ) )

Here is an example query from one of the test cases:

use dataverse test;

write into dataset CustomerOrders1 (
for $c in dataset('Customers1')
for $o in dataset('Orders1')
where $c.cid = $o.cid and $c.age < 21 and $c.total > 50.0
group by $cid := $c.cid decor $cust := $c with $o
return {"cid":$cid, "cust": $cust, "orders": $o}

);

This query will be implemented with a bulkload operator without sorting at
all, which is wrong.

Sattam

Original issue reported on code.google.com by westm...@gmail.com on 25 Jun 2013 at 12:28

GoogleCodeExporter commented 8 years ago
fixed in 
https://code.google.com/p/asterixdb/source/detail?r=35a0f7059149268e9dcda98baeb3
ac1146b42d8e&name=westmann/writestmt

Original comment by westm...@gmail.com on 1 Jul 2013 at 9:13

GoogleCodeExporter commented 8 years ago

Original comment by westm...@gmail.com on 1 Jul 2013 at 9:14

GoogleCodeExporter commented 8 years ago
Only removed the "write into dataset" part of the write statement as I am not 
sure if there is an alternative that can be used for export.

Original comment by westm...@gmail.com on 1 Jul 2013 at 9:15

GoogleCodeExporter commented 8 years ago

Original comment by westm...@gmail.com on 2 Jul 2013 at 1:43