wojciech.kapcia@tigase.net opened 8 years ago
|
|
I've updated section about component startup, but I think we need separate sections about beans and kernel. |
|
I've updated whole section related to component implementation as it was using deprecated solutions. I replaced it with currently recommended design patterns and described it (now it includes Tigase Kernel, data sources usage and a lot of a new stuff). |
|
As usual, comments:
Could you rephrase this sentence? What is same component internal? There is no way to know when all the beans were finally initialised during startup? In Multi-threading there is following passage:
so shouldn't this be included in the list in Lifecycle of component?
|
|
Wojciech Kapcia wrote:
Done, but I'm not sure if is is better.
No. I do not agree that there is anything that requires you to know that. You should be able to work in cases when something is not initialized, and as it is possible to change configuration, enabled/disable components at runtime which makes it impossible to tell if server is initialized at any particular point in time.
I removed that, as in Lifecycle there is already mentioned
Removed.
No, I disagree. There is not default implementation and shouldn't be a default one. Person implementing a module should be required to fill list of features provided by module.
I left it as it was. I updated documentation related to Tigase Kernel.
I have no idea. I left it as it was there. I just updated descriptions related to component implementation and Tigase Kernel.
Yes, there are a few places in which we call
I was updating old code to our current implementation and there was
I've fixed that. For some reason I haven't seen this before. |
|
Andrzej Wójcik wrote:
It is, thank you.
Well, by default it could return empty array, and if someone wants to provide some features this method could simply be overwritten? Looking at implementations roughly half of them return features, and most likely majority of the modules won't have to provide them so this default IMHO is odd.
Yes, but while doing so removing obviously non-existent things makes sense? Especially that DLS is third configuration mechanism from "tigase.xml"…
They are:
At the very least documentation could describe them as deprecated and that usage should be avoided.
It's covered in Afterwards you can assign it to Daniel to fix:
|
|
Wojciech Kapcia wrote:
Fine, I added
OK, I removed it. But whole documentation of Tigase XMPP Server requires clean up, not only this part.
I've updated components to use To do that I needed to change configuration of statistics archivers/loggers, so I updated documentation as well.
I'm assigning this to %wojtek to review changes related to @StatisticsCollector@, it's archivers, etc. Afterwards, if it will finally be ok, assign task to Daniel to fix issue mentioned above. |
|
Andrzej Wójcik wrote:
Yes, I know. I think %Daniel is slowly doing it.
%andrzej.wojcik - lgtm (looks good to me)
Dan, could you proceed with the fixes mentioned above? |
|
Fixed image links, looks like it was just with Dev guide and not all guides as I had thought. |
|
I wanted to check this so we could close this task but currently there is no documentation for 7.2.0 (except of JavaDoc) available at @docs.tigase.org@. Daniel, do you know what is cause of this issue? If not then maybe assign this task to %wojtek so he will check it, as it may be some issue with copying of generated documentation to location used by |
|
Console output shows documentation is built, I think it's to do with the website configuration scraping the wrong directory and finding nothing. Wojciech might have some insight to this. |
|
Daniel Wisnewski wrote:
http://build.tigase.org/jenkins/job/tigase-server-distribution/1183/consoleText Maybe, but probably paths are crooked because I simply copy everything from
Hence only names are copied which results in empty, named blocks. It looks that for some reason you are triggering clean after all building:
which simply deletes all results. Most likely a result of changes in: #5058 leaving on me and setting as blocked by aforementioned issue. |
|
Andrzej Wójcik wrote:
%andrzej.wojcik - I've checked the guides and images are there hence closing the issue. |
|
Eric, can you check why it's impossible to close this issue? |
|
This is really strange, the option of closed isn't present. This has happened in the past. I've closed it in the DB.
|
Type |
Task
|
Priority |
Major
|
Assignee | |
RedmineID |
4890
|
Version |
tigase-server-8.0.0
|
Spent time |
94h 20m
|
Currently we are missing documentation similar to Component Implementation - Lesson 8 - Startup Time (and Development Guide altogether, currently
getDefaults()@/@setProperties()
causes WARNINGS about deprecated methods)