Skip to content

PARQUET-124: normalize path checking to prevent mismatch between URI and ...#79

Closed
chrisalbright wants to merge 3 commits intoapache:masterfrom
chrisalbright:master
Closed

PARQUET-124: normalize path checking to prevent mismatch between URI and ...#79
chrisalbright wants to merge 3 commits intoapache:masterfrom
chrisalbright:master

Conversation

@chrisalbright
Copy link
Contributor

...path

@rdblue
Copy link
Contributor

rdblue commented Nov 19, 2014

Hi @chrisalbright, I just tried to merge this and the script failed because the title doesn't have a ":" after the JIRA number. Could you update it?

@chrisalbright
Copy link
Contributor Author

how embarrasing :/ I have been checking on the JIRA ticket but not here. I'll update the commit message, and re-push, but will that be a problem for you trying to merge?

@rdblue
Copy link
Contributor

rdblue commented Jan 12, 2015

No, it should be fine. Would you mind rebasing on master? That would make it much easier to pull.

@chrisalbright
Copy link
Contributor Author

Ok @rdblue commit message re-written - sorry for the d-e-l-a-a-a-y

@chrisalbright chrisalbright changed the title PARQUET-124 normalize path checking to prevent mismatch between URI and ... PARQUET-124: normalize path checking to prevent mismatch between URI and ... Jan 14, 2015
Conflicts:
	parquet-hadoop/src/test/java/parquet/hadoop/TestParquetFileWriter.java
@julienledem
Copy link
Member

@rdblue did you want to merge?

@asfgit asfgit closed this in e505e1f Jan 30, 2015
@rdblue
Copy link
Contributor

rdblue commented Jan 30, 2015

Merged! Thanks @chrisalbright!

dongche pushed a commit to dongche/incubator-parquet-mr that referenced this pull request Feb 3, 2015
…and ...

...path

Author: Chris Albright <[email protected]>

Closes apache#79 from chrisalbright/master and squashes the following commits:

b1b0086 [Chris Albright] Merge remote-tracking branch 'upstream/master'
9669427 [Chris Albright] PARQUET-124: Adding test (Thanks Ryan Blue) that proves mergeFooters was failing
8e342ed [Chris Albright] PARQUET-124: normalize path checking to prevent mismatch between URI and path
rdblue pushed a commit to rdblue/parquet-mr that referenced this pull request Mar 9, 2015
…and ...

...path

Author: Chris Albright <[email protected]>

Closes apache#79 from chrisalbright/master and squashes the following commits:

b1b0086 [Chris Albright] Merge remote-tracking branch 'upstream/master'
9669427 [Chris Albright] PARQUET-124: Adding test (Thanks Ryan Blue) that proves mergeFooters was failing
8e342ed [Chris Albright] PARQUET-124: normalize path checking to prevent mismatch between URI and path
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