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

Remove COLUMN_AGGREGATION_OPS_METADATA_NAME metadata before serializi… #13304

Merged
merged 1 commit into from Sep 10, 2018

Conversation

Projects
None yet
3 participants
@diosmosis
Member

diosmosis commented Aug 17, 2018

…ng original format since it can have closures.

Remove COLUMN_AGGREGATION_OPS_METADATA_NAME metadata before serializi…
…ng original format since it can have closures.

@diosmosis diosmosis added this to the 3.7.0 milestone Aug 17, 2018

@mattab mattab modified the milestones: 3.7.0, 3.6.1 Sep 1, 2018

@tsteur

This comment has been minimized.

Member

tsteur commented Sep 8, 2018

Should be fine to merge if tests pass.

@diosmosis diosmosis merged commit 3f64755 into 3.x-dev Sep 10, 2018

0 of 2 checks passed

continuous-integration/travis-ci/push The Travis CI build could not complete due to an error
Details
continuous-integration/travis-ci/pr The Travis CI build failed
Details

@diosmosis diosmosis deleted the original-format-closure branch Sep 10, 2018

diosmosis added a commit that referenced this pull request Sep 20, 2018

Remove COLUMN_AGGREGATION_OPS_METADATA_NAME metadata before serializi…
…ng original format since it can have closures. (#13304)

InfinityVoid added a commit to InfinityVoid/matomo that referenced this pull request Oct 11, 2018

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