/regress/sql/foreign_data.sql. Please find attached a updated
patch for the regression tests.
BTW, What do you think about this?
http://archives.postgresql.org/pgsql-hackers/2012-01/msg00229.php
Best regards,
Etsuro Fujita
*** sql/pgsql_fdw.sql.orig 2012-02-13 19:52:08.0 +0900
--- sql
(2012/02/14 15:15), Shigeru Hanada wrote:
(2012/02/13 20:50), Etsuro Fujita wrote:
The patches have been applied, but role-related regression tests failed
in my environment. I fixed it in a similar fashion of
/src/test/regress/sql/foreign_data.sql. Please find attached a updated
patch
(2012/02/14 19:42), Shigeru Hanada wrote:
(2012/02/14 17:40), Etsuro Fujita wrote:
As discussed at
that thread, it would have to change the PlanForeignScan API to let the
FDW generate multiple paths and dump them all to add_path instead
(2012/02/14 23:50), Tom Lane wrote:
Shigeru Hanadashigeru.han...@gmail.com writes:
(2012/02/14 17:40), Etsuro Fujita wrote:
As discussed at
that thread, it would have to change the PlanForeignScan API to let the
FDW generate multiple paths and dump them all to add_path instead of
returning
Hi Hanada-san,
Sorry for the late response.
(2012/02/10 22:05), Shigeru Hanada wrote:
(2011/12/15 11:30), Etsuro Fujita wrote:
(2011/12/14 15:34), Shigeru Hanada wrote:
I think this patch could be marked as Ready for committer with some
minor fixes. Please find attached a revised patch
(2012/02/15 20:50), Etsuro Fujita wrote:
(2012/02/14 23:50), Tom Lane wrote:
(2012/02/14 17:40), Etsuro Fujita wrote:
As discussed at
that thread, it would have to change the PlanForeignScan API to let the
FDW generate multiple paths and dump them all to add_path instead of
returning
foreign data wrapper.
Best regards,
Etsuro Fujita
$ psql postgres
psql (9.2devel)
Type help for help.
postgres=# CREATE ROLE reassign_fdw_user LOGIN SUPERUSER;
CREATE ROLE
postgres=# SET SESSION AUTHORIZATION 'reassign_fdw_user';
SET
postgres=# CREATE ROLE regress_test_role;
CREATE ROLE
postgres
, but as far as I know the guiding principle
here is that superusers always can do whatever they please. Maybe what
you point out is a bug in the behavior (both before and after my patch),
but if so, please raise it separately.
OK. Thanks.
Best regards,
Etsuro Fujita
--
Sent via pgsql-hackers mailing
(2012/03/05 18:21), Shigeru Hanada wrote:
(2012/02/21 20:25), Etsuro Fujita wrote:
Please find attached an updated version of the patch.
This v2 patch can be applied on HEAD cleanly. Compile completed with
only one expected warning of scan.c, and all regression tests for both
core
(2012/03/05 21:00), Etsuro Fujita wrote:
(2012/03/05 18:21), Shigeru Hanada wrote:
(2012/02/21 20:25), Etsuro Fujita wrote:
Please find attached an updated version of the patch.
This v2 patch can be applied on HEAD cleanly. Compile completed with
only one expected warning of scan.c, and all
RelOptInfo.fdw_private seems confusing. How about renaming it
to e.g., RelOptInfo.fdw_state?
Attached is a patch for the draft patch.
Best regards,
Etsuro Fujita
*** a/contrib/file_fdw/file_fdw.c
--- b/contrib/file_fdw/file_fdw.c
***
*** 74,87 static const struct FileFdwOption
are relatively far from the
ones of fdw_private of Path and Plan.
Best regards,
Etsuro Fujita
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
(2012/03/12 13:04), Etsuro Fujita wrote:
(2012/03/09 23:48), Tom Lane wrote:
Etsuro Fujitafujita.ets...@lab.ntt.co.jp writes:
2. IMHO RelOptInfo.fdw_private seems confusing. How about renaming it
to e.g., RelOptInfo.fdw_state?
Why is that better? It seems just as open to confusion
be
present, it may be appropriate to raise an error (just as you
would need to do in the case of a data type mismatch).
Best regards,
Etsuro Fujita
*** a/contrib/file_fdw/file_fdw.c
--- b/contrib/file_fdw/file_fdw.c
***
*** 502,507 fileIterateForeignScan(ForeignScanState
(2012/03/13 15:53), Shigeru HANADA wrote:
(2012/03/12 19:21), Etsuro Fujita wrote:
According to the following documentation on IterateForeignScan() in
50.2. Foreign Data Wrapper Callback Routines, I have created a patch to
support the error handling in file_fdw. Please find attached a patch
key constraints.
Best regards,
Etsuro Fujita
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
don't have
any clear idea to make use of the index information stored in the system
catalogs for better query optimization, but I believe that it's useful
for the ORDER BY push down and/or nestloop-with-inner-parametrized-scan
join optimization.
Thoughts?
Best regards,
Etsuro Fujita
--
Sent via
(2012/03/16 18:58), Heikki Linnakangas wrote:
On 16.03.2012 10:44, Etsuro Fujita wrote:
I have a plan to support 'Create index on foreign table' for 9.3. Here
is my plan.
The index creation is supported for a flat file such as CSV and a remote
table on a RDB e.g., Postgres using CREATE
transformation that defines a one-to-one mapping into the TID space.
Best regards,
Etsuro Fujita
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
(2012/03/17 2:07), David Fetter wrote:
On Fri, Mar 16, 2012 at 11:58:29AM +0200, Heikki Linnakangas wrote:
On 16.03.2012 10:44, Etsuro Fujita wrote:
For a flat file, CREATE INDEX constructs
an index in the same way as an index for a regular table.
For starters, how would you keep
is an updated version of the patch.
Todo:
* More documents
* More tests
Any comments are welcomed.
Thanks,
Best regards,
Etsuro Fujita
copy-popen-20121122.patch
Description: Binary data
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription
find attached an
updated version of the patch.
Thanks,
Best regards,
Etsuro Fujita
copy-popen-20121126.patch
Description: Binary data
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
size if I understand it aright) might
break. Is that OK?
(I would like to think a more simple approach to this optimization.)
Thanks,
Best regards,
Etsuro Fujita
From: pgsql-hackers-ow...@postgresql.org
[mailto:pgsql-hackers-ow...@postgresql.org] On Behalf Of Alexander Korotkov
Sent
From: Tom Lane [mailto:t...@sss.pgh.pa.us]
Etsuro Fujita fujita.ets...@lab.ntt.co.jp writes:
Sorry for the delay. I've reviewed the patch. It was applied
successfully, and it worked well for tests I did including the example
you showed. I think it's worth the work, but I'm not sure you
I'd like to rework on this optimization and submit a patch at the next CF. Is
that okay?
Thanks,
Best regards,
Etsuro Fujita
From: Craig Ringer [mailto:cr...@2ndquadrant.com]
Sent: Friday, January 18, 2013 8:30 PM
To: Alexander Korotkov
Cc: Tom Lane; Etsuro Fujita; pgsql-hackers
command is in progress then program/command is
killed/crashed due to any problem
psql is exiting.
This is a headache. I have no idea how to solve this.
Sorry for the long delay in responding.
Best regards,
Etsuro Fujita
copy-popen-20130123.patch
Description: Binary data
on the expected results of the file. Attached is a patch
fixing the bugs.
Thanks,
Best regards,
Etsuro Fujita
pgsql_fdw_regress.patch
Description: Binary data
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org
Thanks!
Best regards,
Etsuro Fujita
-Original Message-
From: Tom Lane [mailto:t...@sss.pgh.pa.us]
Sent: Friday, July 13, 2012 5:30 AM
To: Etsuro Fujita
Cc: 'Kohei KaiGai'; 'Robert Haas'; pgsql-hackers@postgresql.org
Subject: Re: [HACKERS] WIP Patch: Selective binary conversion
regards,
Etsuro Fujita
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
for this regression
test like src/test/regress/sql/join.sql.
Thanks,
Best regards,
Etsuro Fujita
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
..17633.821 rows=5000 loops=1)
Foreign File: /home/pgsql/relation.csv
Foreign File Size: 48500
Total runtime: 23213.909 ms
(5 rows)
Any comments and suggestions are welcomed.
Thanks,
Best regards,
Etsuro Fujita
file-fdw-pathkeys-0802.patch
Description: Binary data
of the
patch.
Thanks,
Best regards,
Etsuro Fujita
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
Hi Robert,
From: Robert Haas [mailto:robertmh...@gmail.com]
On Thu, Aug 2, 2012 at 7:01 AM, Etsuro Fujita
fujita.ets...@lab.ntt.co.jp wrote:
The following is a comment at fileGetForeignPaths() in contrib/file_fdw.c:
/*
* If data file was sorted, and we knew it somehow, we
From: Robert Haas [mailto:robertmh...@gmail.com]
On Mon, Aug 6, 2012 at 10:33 AM, Tom Lane t...@sss.pgh.pa.us wrote:
Robert Haas robertmh...@gmail.com writes:
On Sun, Aug 5, 2012 at 10:41 PM, Etsuro Fujita
fujita.ets...@lab.ntt.co.jp wrote:
I think file_fdw is useful for managing log
As described in the reference manual for COPY, we should to check file's path
format not to allow relative path. Please find attached a patch.
Thanks,
Best regards,
Etsuro Fujita
copy_from_pathcheck.patch
Description: Binary data
--
Sent via pgsql-hackers mailing list (pgsql-hackers
From: Tom Lane [mailto:t...@sss.pgh.pa.us]
Etsuro Fujita fujita.ets...@lab.ntt.co.jp writes:
As described in the reference manual for COPY, we should to check file's
path
format not to allow relative path. Please find attached a patch.
The argument for disallowing writing to a relative
From: Robert Haas [mailto:robertmh...@gmail.com]
On Thu, Aug 16, 2012 at 2:11 AM, Etsuro Fujita
fujita.ets...@lab.ntt.co.jp wrote:
Agreed. I'd like to withdraw the patch sent in the earlier post, and
propose
to
update the documentation in the COPY reference page. Please find attached
I noticed the syntax of the \copy command in the psql reference page is an old
style. ISTM it's better to update the document. Please find attached a patch.
Thanks,
Best regards,
Etsuro Fujita
psql-copy-ref.patch
Description: Binary data
--
Sent via pgsql-hackers mailing list (pgsql
you. Sorry for the delay.
Best regards,
Etsuro Fujita
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
This is a little patch to fix a typo in a comment.
Best regards,
Etsuro Fujit
commentfix.patch
Description: Binary data
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
Thanks!
Best regards,
Etsuro Fujita
-Original Message-
From: pgsql-hackers-ow...@postgresql.org
[mailto:pgsql-hackers-ow...@postgresql.org] On Behalf Of Kevin Grittner
Sent: Thursday, September 13, 2012 12:40 PM
To: fujita.ets...@lab.ntt.co.jp; pgsql-hackers@postgresql.org
Subject
a variant of
file_fdw, for example a compressed file wrapper and Hadoop HDFS wrapper.)
Thanks,
Best regards,
Etsuro Fujita
copy_options.patch
Description: Binary data
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org
OK I will redesign the function.
Thanks everyone for the advice!
Best regards,
Etsuro Fujita
-Original Message-
From: Andrew Dunstan [mailto:and...@dunslane.net]
Sent: Friday, September 14, 2012 2:27 AM
To: Dimitri Fontaine
Cc: Tom Lane; Etsuro Fujita; pgsql-hackers@postgresql.org
/file_fdw such as compressed file FDW
and Hadoop HDFS FDW, etc., which I think would be useful especially for DWH
environments.
Thanks,
Best regards,
Etsuro Fujita
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org
From: Tom Lane [mailto:t...@sss.pgh.pa.us]
Etsuro Fujita fujita.ets...@lab.ntt.co.jp writes:
Maybe my explanation was insufficient. Let me add one thing to my earlier
explanation. The submitted patch allows the psql \copy instruction to be
executed like:
$ echo '/bin/gunzip -c $1
From: Tom Lane [mailto:t...@sss.pgh.pa.us]
I wrote:
Etsuro Fujita fujita.ets...@lab.ntt.co.jp writes:
I have a question. I think it would be also better to extend the syntax
for the SQL COPY command in the same way, ie,
COPY foo from '/home/pgsql/decompress.sh /home/pgsql/foo.csv.gz
I think we need to update a document on parameterized path in
doc/src/sgml/fdwhandler.sgml. Please find attached a patch.
Thanks,
Best regards,
Etsuro Fujita
begin 666 doc-fdwhandler.patch
M9EF9B M+6=I=!A+V1O8R]SF,OV=M;]F9'=H86YD;5R+G-G;6P@8B]D
M;V,OW)C+W-G;6PO9F1W:%N9QEBYS9VULFEN95X(#AF,3X,C
Thanks!
Best regards,
Etsuro Fujita
-Original Message-
From: Tom Lane [mailto:t...@sss.pgh.pa.us]
Sent: Thursday, October 11, 2012 2:58 AM
To: Etsuro Fujita
Cc: 'PostgreSQL-development'
Subject: Re: [HACKERS] Minor document updates
Etsuro Fujita fujita.ets...@lab.ntt.co.jp
the cost estimate for the path.
Thanks,
Best regards,
Etsuro Fujita
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
This is a little patch to fix a typo in a comment.
Best regards,
Etsuro Fujit
commentfix_20121102.patch
Description: Binary data
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
Sorry, this is not a typo. Please ignore the post.
Best regards,
Etsuro Fujit
-Original Message-
From: pgsql-hackers-ow...@postgresql.org
[mailto:pgsql-hackers-ow...@postgresql.org] On Behalf Of Etsuro Fujita
Sent: Friday, November 02, 2012 2:40 PM
To: 'PostgreSQL-development
ISTM it would be better to update the text about index cost estimation in
indexam.sgml. Please find attached a patch.
Thanks,
Best regards,
Etsuro Fujita
indexam.sgml.patch
Description: Binary data
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your
From: Tom Lane [mailto:t...@sss.pgh.pa.us]
Etsuro Fujita fujita.ets...@lab.ntt.co.jp writes:
ISTM it would be better to update the text about index cost estimation in
indexam.sgml. Please find attached a patch.
I'm not too thrilled with the proposed patch. In the first place, I
don't
From: Tom Lane [mailto:t...@sss.pgh.pa.us]
Etsuro Fujita fujita.ets...@lab.ntt.co.jp writes:
Agreed. However, I am concerned about the next comment in the current code:
/*
* Our generic assumption is that the index pages will be read
* sequentially, so they cost seq_page_cost each
I wrote:
From: Tom Lane [mailto:t...@sss.pgh.pa.us]
I wrote:
Etsuro Fujita fujita.ets...@lab.ntt.co.jp writes:
I have a question. I think it would be also better to extend the syntax
for the SQL COPY command in the same way, ie,
COPY foo from '/home/pgsql/decompress.sh /home
(2012/03/21 4:39), Robert Haas wrote:
On Fri, Mar 16, 2012 at 4:44 AM, Etsuro Fujita
fujita.ets...@lab.ntt.co.jp wrote:
For a flat file, CREATE INDEX constructs an index in the same way as an
index for a regular table.
It seems really weird to have the data half inside the database
a few of those.
OK. I'd like to at first focus on file FDW and Postgres FDW. I'd like
to thank everyone who commented on this topic. Thanks!
Best regards,
Etsuro Fujita
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http
At 22:11 12/03/28 +0900, Shigeru HANADA wrote:
ANALYZE support for foreign tables is proposed by Fujita-san in current
CF, so I'd like to push it.
I updated the patch to the latest HEAD. Please find attached a patch.
Best regards,
Etsuro Fujita
postgresql-analyze-v7.patch
Description
Sorry, I sent this email without noticing Hanada-san' earlier email. So,
please look at Hanada-san's post.
Best regards,
Etsuro Fujita
-Original Message-
From: pgsql-hackers-ow...@postgresql.org
[mailto:pgsql-hackers-ow...@postgresql.org] On Behalf Of Etsuro Fujita
Sent: Thursday, April
Thanks, Hanada-san!
Best regards,
Etsuro Fujita
-Original Message-
From: Shigeru HANADA [mailto:shigeru.han...@gmail.com]
Sent: Friday, April 06, 2012 11:41 AM
To: Etsuro Fujita
Cc: pgsql-hackers@postgresql.org
Subject: Re: [HACKERS] WIP: Collecting statistics on CSV file data
(2012/04
Thanks!
Best regards,
Etsuro Fujita
-Original Message-
From: pgsql-hackers-ow...@postgresql.org
[mailto:pgsql-hackers-ow...@postgresql.org] On Behalf Of Tom Lane
Sent: Saturday, April 07, 2012 4:20 AM
To: Shigeru HANADA
Cc: Etsuro Fujita; pgsql-hackers@postgresql.org
Subject: Re
This is a little patch to fix a typo in contrib/file_fdw.
Best regards,
Etsuro Fujita
file_fdw_typo_fix.patch
Description: Binary data
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql
From: Tom Lane [mailto:t...@sss.pgh.pa.us]
Etsuro Fujita fujita.ets...@lab.ntt.co.jp writes:
This is a little patch to fix a typo in contrib/file_fdw.
I think that comment is fine as-is.
OK, thanks.
Best regards,
Etsuro Fujita
regards, tom lane
--
Sent via
()? I think it is good to just recommend that users
do ANALYZE a foreign table after the validation.)
For the discussion in [3], I've added a new external function
ExecNotNullCheck() and call it from fileIterateForeignScan.
Any comments are welcome.
Best regards,
Etsuro Fujita
[1] https
This is a little patch to fix a typo in file-fdw.sgml
Best regards,
Etsuro Fujita
file_fdw_typo_fix_20120413.patch
Description: Binary data
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql
Thanks!
Best regards,
Etsuro Fujita
-Original Message-
From: Peter Eisentraut [mailto:pete...@gmx.net]
Sent: Saturday, April 14, 2012 7:07 AM
To: Etsuro Fujita
Cc: pgsql-hackers@postgresql.org
Subject: Re: [HACKERS] typo fix
On fre, 2012-04-13 at 17:27 +0900, Etsuro Fujita wrote
Thank you for the review.
-Original Message-
From: pgsql-hackers-ow...@postgresql.org
[mailto:pgsql-hackers-ow...@postgresql.org] On Behalf Of Andrew Dunstan
Sent: Friday, April 13, 2012 9:16 PM
To: Shigeru HANADA
Cc: Etsuro Fujita; pgsql-hackers@postgresql.org
Subject: Re
I updated the patch. Attached is an updated version of the patch.
Changes:
* fix a bug in fileGetOptions()
* rename the validation option and its code to validate_data_file
* clean up
Best regards,
Etsuro Fujita
-Original Message-
From: pgsql-hackers-ow...@postgresql.org
column
data types as well as all kinds of constraints.
I appreciate your comments.
Best regards,
Etsuro Fujita
[1] http://homepages.cwi.nl/~idreos/NoDBsigmod2012.pdf
[2] https://commitfest.postgresql.org/action/patch_view?id=822
file_fdw_sel_bin_conv_v1.patch
Description: Binary data
--
Sent via
-Original Message-
From: David Fetter [mailto:da...@fetter.org]
Sent: Wednesday, May 09, 2012 9:25 AM
To: Etsuro Fujita
Cc: pgsql-hackers@postgresql.org
Subject: Re: [HACKERS] WIP Patch: Selective binary conversion of CSV file
foreign tables
On Tue, May 08, 2012 at 08:26:02PM
-Original Message-
From: Robert Haas [mailto:robertmh...@gmail.com]
Sent: Friday, May 11, 2012 1:36 AM
To: Etsuro Fujita
Cc: pgsql-hackers@postgresql.org
Subject: Re: [HACKERS] WIP Patch: Selective binary conversion of CSV file
foreign tables
On Tue, May 8, 2012 at 7:26 AM
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
with that of a full plain table scan. So, in such a
case a full hash aggregation is not so time consuming. Am I wrong?
Best regards,
Etsuro Fujita
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
Hi,
-Original Message-
From: Robert Haas [mailto:robertmh...@gmail.com]
Sent: Tuesday, June 19, 2012 3:12 AM
To: Ants Aasma
Cc: Etsuro Fujita; Jay Levitt; Tom Lane; PostgreSQL-development; Francois
Deliege
Subject: Re: [HACKERS] [PATCH] Lazy hashaggregate when no aggregation
Hi KaiGai-san,
Thank you for the review.
-Original Message-
From: pgsql-hackers-ow...@postgresql.org
[mailto:pgsql-hackers-ow...@postgresql.org] On Behalf Of Kohei KaiGai
Sent: Wednesday, June 20, 2012 1:26 AM
To: Etsuro Fujita
Cc: Robert Haas; pgsql-hackers@postgresql.org
Subject
I rebased the patch to current head. Attached is an updated version of the
patch.
Best regards,
Etsuro Fujita
-Original Message-
From: pgsql-hackers-ow...@postgresql.org
[mailto:pgsql-hackers-ow...@postgresql.org] On Behalf Of Etsuro Fujita
Sent: Tuesday, April 17, 2012 2:40 PM
Hi Kaigai-san,
-Original Message-
From: Kohei KaiGai [mailto:kai...@kaigai.gr.jp]
Sent: Monday, June 25, 2012 9:49 PM
To: Etsuro Fujita
Cc: Robert Haas; pgsql-hackers@postgresql.org
Subject: Re: [HACKERS] WIP Patch: Selective binary conversion of CSV file
foreign tables
Fujita
Hi Kaigai-san,
-Original Message-
From: Kohei KaiGai [mailto:kai...@kaigai.gr.jp]
Sent: Tuesday, June 26, 2012 11:05 PM
To: Etsuro Fujita
Cc: Robert Haas; pgsql-hackers@postgresql.org
Subject: Re: [HACKERS] WIP Patch: Selective binary conversion of CSV file
foreign tables
2012/6
Hi,
-Original Message-
From: Robert Haas [mailto:robertmh...@gmail.com]
Sent: Wednesday, June 27, 2012 5:09 AM
To: Etsuro Fujita
Cc: Ants Aasma; Jay Levitt; Tom Lane; PostgreSQL-development; Francois Deliege
Subject: Re: [HACKERS] [PATCH] Lazy hashaggregate when no aggregation
Hi Ants,
-Original Message-
From: Ants Aasma [mailto:a...@cybertec.at]
Sent: Wednesday, June 27, 2012 9:23 PM
To: Robert Haas
Cc: Etsuro Fujita; Jay Levitt; Tom Lane; PostgreSQL-development; Francois
Deliege
Subject: Re: [HACKERS] [PATCH] Lazy hashaggregate when no aggregation
Through the work on the patch [1], I had a question about the psql \copy
command. We are permitted 1) but not permitted 2):
1) \copy foo from stdin ;
2) \copy foo from stdin;
Is this intentional? I think it would be better to allow for 2). Attached is a
patch.
Thanks,
Best regards,
Etsuro
I found a comment typo. Please find attached a patch.
Best regards,
Etsuro Fujita
comment_typo.patch
Description: Binary data
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
Sorry, I found one more typo. Attached is a patch.
Thanks,
Best regards,
Etsuro Fujita
-Original Message-
From: Magnus Hagander [mailto:mag...@hagander.net]
Sent: Friday, February 08, 2013 7:47 PM
To: Etsuro Fujita
Cc: PostgreSQL-development
Subject: Re: [HACKERS] Comment typo
.
Sorry for the long delay.
Best regards,
Etsuro Fujita
With Regards,
Amit Kapila.
copy-popen-20130220.patch
Description: Binary data
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
Hi Amit,
Thank you for your careful review!
-Original Message-
From: Amit Kapila [mailto:amit.kap...@huawei.com]
Sent: Friday, February 22, 2013 7:18 PM
To: 'Etsuro Fujita'; 'pgsql-hackers'
Subject: RE: [HACKERS] Review : Add hooks for pre- and post-processor
executables for COPY
, NULL, ',
0, false, false, pset.encoding);
if (!token)
goto error;
strtokx() in the above should be called in the following way:
token = strtokx(NULL, whitespace, ;, ',
0, false, false, pset.encoding);
Thanks,
Best regards,
Etsuro Fujita
I ran into a typo in the reference page on the SELECT command. Please find
attached a patch.
Best regards,
Etsuro Fujita
typo_fix_20130312.patch
Description: Binary data
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http
readability, but I'm not sure it's worth the
work for this optimization, though I think I'm missing something.
Thanks,
Best regards,
Etsuro Fujita
unused-targets-2.patch
Description: Binary data
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your
)
Filter: ((aid = 0) AND (aid = 1))
Foreign File: /home/pgsql/sample_csv_data1.csv
Foreign File Size: 54396
Total runtime: 15114.480 ms
(13 rows)
Best regards,
Etsuro Fujita
diff -crNB original/postgresql-9.1beta1/contrib/file_fdw/file_fdw.c
changed
integer
ALTER [COLUMN] column SET ( n_distinct = val ) (n_distinct only)
ALTER [COLUMN] column RESET ( n_distinct )
* reflection of the force_not_null info in acquiring sample rows
* documentation
Best regards,
Etsuro Fujita
*** a/contrib/file_fdw/file_fdw.c
--- b/contrib/file_fdw
that feature for future work.
(But this is BTW. I'm interested in developing CREATE FOREIGN INDEX.
I've examined whether there are discussions about the design and
implementation of it in the archive, but could not find information. If
you know anything, please tell me.)
Best regards,
Etsuro Fujita
into the virtual index interface from Informix.
Thank you for the information.
We might want to start a wiki page on this.
Yeah, I think it might be better to add information to the SQL/MED wiki
page:
http://wiki.postgresql.org/wiki/SQL/MED
Best regards,
Etsuro Fujita
--
Sent via pgsql
or somewhere in core.
- It should be mentioned in a document that foreign tables are not
analyzed automatically because they are read-only.
OK. I'll revise.
Regards,
Best regards,
Etsuro Fujita
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your
of foreign tables...
The patch does not analyze on foreign tables automatically. (The issue
of auto-analyze on foreign tables has been discussed. Please refer to [1].)
[1] http://archives.postgresql.org/pgsql-hackers/2011-09/msg00992.php
Best regards,
Etsuro Fujita
--
Sent via pgsql-hackers
(2011/11/07 20:26), Shigeru Hanada wrote:
(2011/10/20 18:56), Etsuro Fujita wrote:
I revised the patch according to Hanada-san's comments. Attached is the
updated version of the patch.
Changes:
* pull up of logging analyzing foo.bar
* new vac_update_relstats always called
those tuples as dead tuples. This is
for the consistency with NOT NULL constrain. (But I don't know why
fileIterateForeignScan routine allows such dead tuples. I may have
missed something.)
Best regards,
Etsuro Fujita
(2011/11/18 21:00), Shigeru Hanada wrote:
(2011/11/18 16:25), Etsuro Fujita
for foreign index scan on CSV file data that I plan
to propose in the next CF. So, it is meaningless for now. I'm sorry.
I will fix it at the next version of the patch so that they don't have to.
Best regards,
Etsuro Fujita
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org
and widely applicable to the other DBMSs, while it,
of course, only works once we have statistics and/or index information
for foreign tables. But AFAIK we eventually want to have those, so I'd
like to propose to use the proposed approach until that time.
Best regards,
Etsuro Fujita
--
Sent via pgsql
(2011/11/28 20:50), Shigeru Hanada wrote:
(2011/11/25 17:27), Etsuro Fujita wrote:
So, I think it might be better to estimate
such costs by pgsql_fdw itself without EXPLAINing on the assumption that
a remote postgres server has the same abilities for query
Hi Hanada-san,
I updated the patch. Please find attached a patch.
Best regards,
Etsuro Fujita
(2011/11/18 21:00), Shigeru Hanada wrote:
(2011/11/18 16:25), Etsuro Fujita wrote:
Thank you for your testing. I updated the patch according to your
comments. Attached is the updated version
1 - 100 of 949 matches
Mail list logo