asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Till Westmann (Code Review)" <do-not-re...@asterixdb.incubator.apache.org>
Subject Change in asterixdb[master]: Load parser from classpath in ParserFactoryProvider
Date Tue, 10 Jan 2017 22:09:31 GMT
Till Westmann has posted comments on this change.

Change subject: Load parser from classpath in ParserFactoryProvider
......................................................................


Patch Set 3: Code-Review+2

I think that the integration should be easy. If you put the CAP parser into a separate module
in BAD's asterix-opt with it's own service configuration file, maven should build a jar and
put it on the classpath. When running ParserFactoryProvider.initFactories should find the
configuration file in that jar and add the factory to the factory provider.

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1416
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I2ac039fe3daaf0636cf004289bd0c8a3229197a9
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Xikui Wang <xkkwww@gmail.com>
Gerrit-Reviewer: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Till Westmann <tillw@apache.org>
Gerrit-Reviewer: Xikui Wang <xkkwww@gmail.com>
Gerrit-HasComments: No

Mime
View raw message