Skip to content

ARROW-895: Fix lastSet in fillEmpties() and copyFrom()#601

Closed
StevenMPhillips wants to merge 1 commit into
apache:masterfrom
StevenMPhillips:fillEmpties4
Closed

ARROW-895: Fix lastSet in fillEmpties() and copyFrom()#601
StevenMPhillips wants to merge 1 commit into
apache:masterfrom
StevenMPhillips:fillEmpties4

Conversation

@StevenMPhillips

Copy link
Copy Markdown
Contributor

No description provided.

@asfgit asfgit closed this in 1a73c35 Apr 25, 2017
jeffknupp pushed a commit to jeffknupp/arrow that referenced this pull request Jun 3, 2017
Author: Steven Phillips <steven@dremio.com>

Closes apache#601 from StevenMPhillips/fillEmpties4 and squashes the following commits:

4707673 [Steven Phillips] ARROW-895: Fix lastSet in fillEmpties() and copyFrom()
pribor pushed a commit to GlobalWebIndex/arrow that referenced this pull request Oct 24, 2025
Author: Steven Phillips <steven@dremio.com>

Closes apache#601 from StevenMPhillips/fillEmpties4 and squashes the following commits:

4707673 [Steven Phillips] ARROW-895: Fix lastSet in fillEmpties() and copyFrom()
pribor pushed a commit to GlobalWebIndex/arrow that referenced this pull request Oct 24, 2025
…ache#602)

### Rationale for this change
Multiple threads can attempt to create the same llvm expression in
Gandiva. This isn't allowed with the new JIT compiler, so synchronizing
will prevent this scenario.

### What changes are included in this PR?
Synchronize some methods to avoid adding duplicate llvm expressions.

### Are these changes tested?
Yes, through unit tests in Gandiva.

### Are there any user-facing changes?
No.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants