-
Notifications
You must be signed in to change notification settings - Fork 82
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added support for parameterized PailStructures #17
Conversation
Don't ignore git file!
Conflicts: src/jvm/backtype/hadoop/Consolidator.java src/jvm/backtype/hadoop/datastores/DefaultTimeSliceStructure.java src/jvm/backtype/hadoop/pail/AbstractPail.java src/jvm/backtype/hadoop/pail/PailSpec.java src/jvm/backtype/hadoop/pail/PailStructure.java test/jvm/backtype/hadoop/pail/PailTest.java
serialization behavior (that serializes only a single field and not the containing tuple) and the new behavior (that serializes the selected fields of each tuple as a tuple).
👍 |
@sorenmacbeth, what do you think? big change here. |
@sorenmacbeth Please sort this out. the number and kind of pull requests already show that this kind of feature is most desirable for api users. 😄 |
The pull request looks like a dead end as it seems that author has developed it into a person project. Whole code organization is changed even the Build files for scala are deleted. although this could have been the most elegant way to support dynamic pail structures. 😢 commits can be cherry picked but, seems like rewriting support for Jackson would be more easier! |
Yes, I wrote the pull request. After no action was taken, I dealt with the many other issues that arose. Cherry picking the Jackson support would be easy.... — On Fri, Sep 13, 2013 at 10:01 PM, kul [email protected] wrote:
|
This declares a PailStructure that takes a single argument, "template", with value "${type}".
Here is the corresponding constructor with Jackson 2.0 annotations: