> On Aug. 7, 2014, 6:37 a.m., shwethags wrote: > > addons/designer/actions/src/main/java/org/apache/falcon/designer/action/serde/impl/EmailActionSerde.java, > > line 40 > > <https://reviews.apache.org/r/24311/diff/4/?file=653669#file653669line40> > > > > Why don't you use https://sites.google.com/site/gson/gson-user-guide? > > > > All these exceptions should be thrown back instead of returning null > > and handling null later. > > > > Isn't serde() common that they can be moved to base class, say > > configuration
Not moving to gson at this point of time. Jackson seem to do the job.. serdes are removed - samar ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24311/#review49872 ----------------------------------------------------------- On Aug. 8, 2014, 11:59 a.m., samar kumar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24311/ > ----------------------------------------------------------- > > (Updated Aug. 8, 2014, 11:59 a.m.) > > > Review request for Falcon. > > > Bugs: FALCON-553 > https://issues.apache.org/jira/browse/FALCON-553 > > > Repository: falcon-git > > > Description > ------- > > Base line the code and https://issues.apache.org/jira/browse/FALCON-553 in > progress > > > Diffs > ----- > > addons/designer/actions/pom.xml 7d5afbb > > addons/designer/actions/src/main/java/org/apache/falcon/designer/action/configuration/EmailActionConfiguration.java > PRE-CREATION > > addons/designer/actions/src/main/java/org/apache/falcon/designer/primitive/action/EmailAction.java > PRE-CREATION > addons/designer/core/pom.xml ddd8814 > > addons/designer/core/src/main/java/org/apache/falcon/designer/configuration/ActionConfiguration.java > PRE-CREATION > > addons/designer/core/src/main/java/org/apache/falcon/designer/configuration/Configuration.java > dba908a > > addons/designer/core/src/main/java/org/apache/falcon/designer/configuration/FlowConfig.java > PRE-CREATION > > addons/designer/core/src/main/java/org/apache/falcon/designer/configuration/SerdeException.java > PRE-CREATION > > addons/designer/core/src/main/java/org/apache/falcon/designer/configuration/TransformConfiguration.java > PRE-CREATION > > addons/designer/core/src/main/java/org/apache/falcon/designer/primitive/Action.java > c40e462 > > addons/designer/core/src/main/java/org/apache/falcon/designer/primitive/Primitive.java > aa2b988 > > addons/designer/core/src/main/java/org/apache/falcon/designer/primitive/Transform.java > 72cf988 > > addons/designer/core/src/main/java/org/apache/falcon/designer/storage/impl/HDFSStorage.java > PRE-CREATION > > addons/designer/core/src/test/java/org/apache/falcon/designer/storage/impl/HDFSStorageTest.java > PRE-CREATION > addons/designer/flows/pom.xml ce706a3 > > addons/designer/flows/src/main/java/org/apache/falcon/designer/primitive/Flow.java > PRE-CREATION > addons/designer/pom.xml 3e1a98a > > Diff: https://reviews.apache.org/r/24311/diff/ > > > Testing > ------- > > > Thanks, > > samar kumar > >
