Skip to content

ARROW-1868: [Java] Cleanup usage of Types.MinorType to MinorType#1831

Closed
BryanCutler wants to merge 1 commit into
apache:masterfrom
BryanCutler:java-MinorType-cleanup-ARROW-1463
Closed

ARROW-1868: [Java] Cleanup usage of Types.MinorType to MinorType#1831
BryanCutler wants to merge 1 commit into
apache:masterfrom
BryanCutler:java-MinorType-cleanup-ARROW-1463

Conversation

@BryanCutler

Copy link
Copy Markdown
Member

Cleanup usage specifying Types.MinorType to just MinorType in vector classes and tests, and removed unused imports.

@icexelloss icexelloss left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@BryanCutler

Copy link
Copy Markdown
Member Author

Thanks @icexelloss! Does this cleanup look ok from your end @siddharthteotia ?

@BryanCutler

Copy link
Copy Markdown
Member Author

I'll commit this later today if no more comments

@siddharthteotia siddharthteotia left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

+1, Thanks, @BryanCutler

@BryanCutler BryanCutler changed the title ARROW-1463: [Java] Cleanup usage of Types.MinorType to MinorType ARROW-1868: [Java] Cleanup usage of Types.MinorType to MinorType Apr 4, 2018
@BryanCutler

Copy link
Copy Markdown
Member Author

Merged to master, thanks @icexelloss and @siddharthteotia

pribor pushed a commit to GlobalWebIndex/arrow that referenced this pull request Oct 24, 2025
Cleanup usage specifying Types.MinorType to just MinorType in vector classes and tests, and removed unused imports.

Author: Bryan Cutler <cutlerb@gmail.com>

Closes apache#1831 from BryanCutler/java-MinorType-cleanup-ARROW-1463 and squashes the following commits:

4b9560e <Bryan Cutler> Cleanup usage of Types.MinorType to MinorType in vectors and tests
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.

3 participants