Skip to content

Conversation

@JaroslavTulach
Copy link
Member

@JaroslavTulach JaroslavTulach commented Jan 14, 2026

Pull Request Description

from Standard.Base import all
from Standard.Table import all
from Standard.Tableau import all

main =
    node1 = Data.read '/tmp/x.hyper'
    node2 = node1.read 'Extract'
    node3 = node2.set (expr 'year([Date])') "Year"
    node3.get "Year" . to_vector

in dual JVM (mock) mode:

sbt:enso> runEngineDistribution --vm.D=polyglot.enso.classLoading=Standard.Tableau:guest,hosted --run hyper.enso
Error:Argument_Mismatch Unsupported type: class com.oracle.truffle.polyglot.PolyglotMapAndFunction (expected date/time type).

Checklist

Please ensure that the following checklist has been satisfied before submitting the PR:

  • All code follows the
    Java,
  • Unit tests have been written where possible.

@JaroslavTulach JaroslavTulach self-assigned this Jan 14, 2026
@JaroslavTulach JaroslavTulach added the -libs Libraries: New libraries to be implemented label Jan 14, 2026
@JaroslavTulach JaroslavTulach added the CI: No changelog needed Do not require a changelog entry for this PR. label Jan 14, 2026
offheapBuffer = buf;
validitySet = new ImmutableBitSet(validity, data.length);
}
if (offheapBuffer == null && getType() instanceof DateType) {
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jdunkerley, it would be more efficient if the DateBuilder was creating a specialized storage backed by ByteBuffer rather than converting LocalDate[] into the ByteBuffer when requested. We'd be allocating way less objects by doing so...

@JaroslavTulach JaroslavTulach added the CI: Ready to merge This PR is eligible for automatic merge label Jan 14, 2026
@mergify mergify bot merged commit d99dff2 into develop Jan 14, 2026
78 of 82 checks passed
@mergify mergify bot deleted the wip/jtulach/LocalDateArrowStorage14607 branch January 14, 2026 16:27
@enso-bot
Copy link

enso-bot bot commented Jan 15, 2026

Jaroslav Tulach reports a new STANDUP for yesterday (2026-01-14):

Progress: .

GitHub
Pull Request Description

close #14142

Important Notes

Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

The documentation has been updated, if ...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

-libs Libraries: New libraries to be implemented CI: No changelog needed Do not require a changelog entry for this PR. CI: Ready to merge This PR is eligible for automatic merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants