ARROW-411: [Java] Move compactor functions in Integration to a separate Validator module #267

Closed
wants to merge 1 commit into
from

Projects

None yet

3 participants

@icexelloss
Contributor

No description provided.

@icexelloss icexelloss changed the title from Move compactor functions in Integration to a separate Validator module to [ARROW-411]Move compactor functions in Integration to a separate Validator module Jan 3, 2017
@icexelloss icexelloss changed the title from [ARROW-411]Move compactor functions in Integration to a separate Validator module to [ARROW-411] Move compactor functions in Integration to a separate Validator module Jan 3, 2017
@icexelloss icexelloss changed the title from [ARROW-411] Move compactor functions in Integration to a separate Validator module to ARROW-411: [Java] Move compactor functions in Integration to a separate Validator module Jan 3, 2017
@icexelloss icexelloss ARROW-411: Move compator functions in Integration to a separate Valid…
…ator moduleO
b4e86c5
@BryanCutler
Contributor
BryanCutler commented Jan 3, 2017 edited

Just wondering if it might fit better under package org.apache.arrow.tools, but I don't know the Java layout too well. Besides that, LGTM
Nevermind that, I see there was already discussion in the JIRA about the location, this is fine with me.

@wesm
wesm approved these changes Jan 5, 2017 View changes

+1, LGTM

@asfgit asfgit pushed a commit that closed this pull request Jan 5, 2017
@icexelloss @wesm icexelloss + wesm ARROW-411: [Java] Move compactor functions in Integration to a separa…
…te Validator module

Author: Li Jin <ice.xelloss@gmail.com>

Closes #267 from icexelloss/validator and squashes the following commits:

b4e86c5 [Li Jin] ARROW-411: Move compator functions in Integration to a separate Validator moduleO
9513ca7
@asfgit asfgit closed this in 9513ca7 Jan 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment