Subject: Re: the ScriptingMojo
To: Kartik Ohri <kartikohri13@gmail.com>
References: 
 <CAASLQ4Mva7dAXiS_c9OLhV4VzNHF=XNKJosuS5gtZTB6b4TW1Q@mail.gmail.com>
 <5F2EC4F4.3080608@anastigmatix.net>
 <CAASLQ4M3pR+ZL+uPvcQ-dAEL2KdH_PGe-_pBhsjvOgN5qvf+aA@mail.gmail.com>
 <5F31EFEC.1080609@anastigmatix.net>
 <CAASLQ4NU9v=nuLDG-LneC68LuqKRXchFckU7mCZT9YQDNKqwvg@mail.gmail.com>
 <5F368380.9070203@anastigmatix.net>
 <CAASLQ4NgH+tXbdUSA8xDh2ct3gAS3xa8V7=E0gcYd+d=m8kujw@mail.gmail.com>
 <5F368A99.6070709@anastigmatix.net> <5F36965C.60707@anastigmatix.net>
 <5F3A7DC6.8090905@anastigmatix.net>
 <CAASLQ4PEgo5Nj0EVsOfGoB96dQFXfFYxrrB0rS1wdad7x_E5fA@mail.gmail.com>
 <CAASLQ4OowXOmnbbbT0Wh1ZkB4h=6JQfC61ojmN07EYnO0QCcow@mail.gmail.com>
 <5F3D32E9.80504@anastigmatix.net> <5F3D36BE.70703@anastigmatix.net>
 <5F3D38F8.7050907@anastigmatix.net> <5F3D3A75.5010304@anastigmatix.net>
 <5F3D3D2E.4070109@anastigmatix.net> <5F3D43BD.4020900@anastigmatix.net>
 <5F3DA48D.7090408@anastigmatix.net>
 <CAASLQ4NGoruXs1+fNsXnsic7cRjqceJ-PiYofG-OMPUWnxDjWA@mail.gmail.com>
Cc: pljava-dev@lists.postgresql.org
From: Chapman Flack <chap@anastigmatix.net>
Message-ID: <5F4172F9.4060608@anastigmatix.net>
Date: Sat, 22 Aug 2020 15:33:13 -0400
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101
 Thunderbird/38.7.0
MIME-Version: 1.0
In-Reply-To: 
 <CAASLQ4NGoruXs1+fNsXnsic7cRjqceJ-PiYofG-OMPUWnxDjWA@mail.gmail.com>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 08/22/20 06:15, Kartik Ohri wrote:
> Hi!
> I am working adding these changes but I am particularly stuck at this one.
> It seems that Nashorn and Graal do not support SAM inside SAM or I am doing
> this the wrong way. Please see https://www.ideone.com/cQVGuB and
> https://www.ideone.com/1x7Ah7. The former one fails at runtime but the
> latter completes execution. (Here I have done a Consumer<Function, ?>
> whereas Function<Consumer, ?> will be required for our use case, but both
> fail at runtime anyways). This is the commit (
> https://github.com/amCap1712/pljava/commit/136a8e879951432ec47faeb74e2a86e848481e67)
> with changes that fails. I have been able to work around the issue using
> the approach here
> https://github.com/amCap1712/pljava/commit/d5977c5cf386758fb2d9e56524b05f9549cefe31

Just getting a look at this now. In the (failing) 136a commit, it appears
to me that nashorn fails but graal is happy. Is that what you saw?

Graal fails at a different spot, on the Files.walk().parallel()...
invocation in the script, but that's clearly a different issue;
neither script engine wants to be shared across threads. A sequential
walk is adequate here; this is just a build process that runs once,
so shaving milliseconds isn't essential.

Ok, I removed the parallel() and now graal also complains about the SAM
cast. This puzzles me because I thought both engines were intended to
support that, so I'll keep playing with it a bit.

Regards,
-Chap