peter-lawrey/Java-Chronicle

Make Java Chronicle OSGi ready

lburgazzoli opened this issue · 6 comments

lburgazzoli commented

Would it be possible to build Java Chronicle as OSGi bundle?

peter-lawrey commented

It was an OSGi bundle, it appears that a refactor done earlier broke this.
I will try to fix this this week.

On 27 August 2013 21:23, lburgazzoli notifications@github.com wrote:

Would it be possible to build Java Chronicle as OSGi bundle?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/35
.

lburgazzoli commented

I did check the version 1.8 and now it is OSGi compliant but it does not export the package com.higherfrequencytrading.chronicle.impl which is required by other bundles (example reactor https://github.com/reactor/reactor):

[ 197] [Installed  ] [            ] [   80] chronicle (1.8.0)
karaf@lb> export 197
Command not found: export
karaf@lb> exports 197
    ID Packages                                
   197 com.higherfrequencytrading.chronicle; version=1.8.0
   197 com.higherfrequencytrading.chronicle.datamodel; version=1.8.0
   197 com.higherfrequencytrading.chronicle.math; version=1.8.0
   197 com.higherfrequencytrading.chronicle.tcp; version=1.8.0
   197 com.higherfrequencytrading.chronicle.tcp.gw; version=1.8.0
   197 com.higherfrequencytrading.chronicle.tools; version=1.8.0

Is that intentional?

peter-lawrey commented

I have checked that the impl package is in the MANIFEST file now. I have
released 1.8.1

On 26 September 2013 05:30, lburgazzoli notifications@github.com wrote:

I did check the version 1.8 and now it is OSGi compliant but it does not
export the package com.higherfrequencytrading.chronicle.impl which is
required by other bundles (example reactor
https://github.com/reactor/reactor):

[ 197] [Installed ] [ ] [ 80] chronicle (1.8.0)
karaf@lb> export 197
Command not found: export
karaf@lb> exports 197
ID Packages
197 com.higherfrequencytrading.chronicle; version=1.8.0
197 com.higherfrequencytrading.chronicle.datamodel; version=1.8.0
197 com.higherfrequencytrading.chronicle.math; version=1.8.0
197 com.higherfrequencytrading.chronicle.tcp; version=1.8.0
197 com.higherfrequencytrading.chronicle.tcp.gw; version=1.8.0
197 com.higherfrequencytrading.chronicle.tools; version=1.8.0

Is that intentional?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/35#issuecomment-25163519
.

peter-lawrey commented

BTW You might find that Chronicle 2.0.1 is faster than 1.8.1. ;)

https://github.com/OpenHFT/Java-Chronicle

On 26 September 2013 06:30, Peter Lawrey peter.lawrey@gmail.com wrote:

I have checked that the impl package is in the MANIFEST file now. I have
released 1.8.1

On 26 September 2013 05:30, lburgazzoli notifications@github.com wrote:

I did check the version 1.8 and now it is OSGi compliant but it does not
export the package com.higherfrequencytrading.chronicle.impl which is
required by other bundles (example reactor
https://github.com/reactor/reactor):

[ 197] [Installed ] [ ] [ 80] chronicle (1.8.0)
karaf@lb> export 197
Command not found: export
karaf@lb> exports 197
ID Packages
197 com.higherfrequencytrading.chronicle; version=1.8.0
197 com.higherfrequencytrading.chronicle.datamodel; version=1.8.0
197 com.higherfrequencytrading.chronicle.math; version=1.8.0
197 com.higherfrequencytrading.chronicle.tcp; version=1.8.0
197 com.higherfrequencytrading.chronicle.tcp.gw; version=1.8.0
197 com.higherfrequencytrading.chronicle.tools; version=1.8.0

Is that intentional?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/35#issuecomment-25163519
.

lburgazzoli commented

thx, I'll check it out.
btw I see that you have also started working on a FIX engine, interesting

peter-lawrey commented

My next task is file rotation support in Chronicle 2.1

On 26 September 2013 06:49, lburgazzoli notifications@github.com wrote:

thx, I'll check it out.
btw I see that you have also started working on a FIX engine, interesting

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/35#issuecomment-25168614
.