Skip to content

Java 7 deprecation - #1640

Merged
lossyrob merged 4 commits into
locationtech:masterfrom
pomadchin:feature/java-8
Oct 6, 2016
Merged

lossyrob merged 4 commits into
locationtech:masterfrom
pomadchin:feature/java-8

Conversation

@pomadchin

@pomadchin pomadchin commented Sep 28, 2016 •

Copy link
Copy Markdown
Member
  • Java 8 support only
  • Removed joda time
  • Removed de.javakaffe
  • Test on Landsat EMR demo

@pomadchin pomadchin changed the title Java 7 deprecation [WIP] Java 7 deprecation Sep 28, 2016
@pomadchin pomadchin changed the title [WIP] Java 7 deprecation Java 7 deprecation Sep 29, 2016
@pomadchin

pomadchin commented Sep 29, 2016 •

Copy link
Copy Markdown
Member Author

Still need to double check following serializers usage (from de.javakaffe):

kryo.register( Arrays.asList( "" ).getClass, new ArraysAsListSerializer )       
kryo.register( Collections.EMPTY_LIST.getClass, new CollectionsEmptyListSerializer() )      
kryo.register( Collections.EMPTY_MAP.getClass, new CollectionsEmptyMapSerializer() )        
kryo.register( Collections.EMPTY_SET.getClass, new CollectionsEmptySetSerializer() )        
kryo.register( Collections.singletonList( "" ).getClass, new CollectionsSingletonListSerializer )       
kryo.register( Collections.singleton( "" ).getClass, new CollectionsSingletonSetSerializer )        
kryo.register( Collections.singletonMap( "", "" ).getClass, new CollectionsSingletonMapSerializer )

UnmodifiableCollectionsSerializer.registerSerializers( kryo )       
SynchronizedCollectionsSerializer.registerSerializers( kryo )

@lossyrob

lossyrob commented Oct 5, 2016

Copy link
Copy Markdown
Member

Is this WIP because of the double checking?

@pomadchin

Copy link
Copy Markdown
Member Author

@lossyrob can be merged (: I use this branch for other prs locally

@lossyrob
lossyrob merged commit 9860220 into locationtech:master Oct 6, 2016
@lossyrob lossyrob added this to the 1.0 milestone Oct 18, 2016
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.

2 participants