Skip to content
This repository has been archived by the owner on Dec 13, 2023. It is now read-only.

Jxu/cassandra serde #3144

Merged
merged 2 commits into from
Aug 5, 2022
Merged

Jxu/cassandra serde #3144

merged 2 commits into from
Aug 5, 2022

Conversation

jxu-nflx
Copy link
Contributor

@jxu-nflx jxu-nflx commented Aug 2, 2022

Pull Request type

  • Bugfix
  • Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes (Please run ./gradlew generateLock saveLock to refresh dependencies)
  • WHOSUSING.md
  • Other (please describe):

NOTE: Please remember to run ./gradlew spotlessApply to fix any format violations.

Changes in this PR

  • Ignore empty workflowtask field when saving to Cassandra
  • Add afterburner module to optimize json serializers and deserializers

Review each commit separately for better visibility of the changes

Describe the new behavior from this PR, and why it's needed
Issue #

Alternatives considered

Describe alternative implementation you have considered

@jxu-nflx jxu-nflx force-pushed the jxu/cassandra_serde branch 3 times, most recently from 79c1a1b to 599fae3 Compare August 4, 2022 21:52
@jxu-nflx jxu-nflx force-pushed the jxu/cassandra_serde branch from 599fae3 to bee8b85 Compare August 4, 2022 22:09
@jxu-nflx jxu-nflx requested a review from apanicker-nflx August 4, 2022 22:22
@jxu-nflx jxu-nflx merged commit ab14727 into main Aug 5, 2022
@jxu-nflx jxu-nflx deleted the jxu/cassandra_serde branch August 5, 2022 20:09
pmchung pushed a commit to routific/conductor that referenced this pull request Sep 6, 2023
* Ignore empty field in json serialization

* Add afterburner module to optimize json serializers and deserializers
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants