Hi all!
The PRs just get merged.
Now on to the next step ->
https://github.com/apache/incubator-kie-issues/issues/2352

Thanks again!

Cheers
Gabriele

On Wed, Jun 24, 2026 at 6:52 AM Chinchu P Shaji via dev <[email protected]>
wrote:

> Hi @GABRIELE CARDOSI<mailto:[email protected]>
>
> Thanks for driving this initiative — it’s a significant improvement.
> Configurable properties are now easier to identify and work with, which
> reduces errors and speeds up development. The inclusion of metadata such as
> field type and default values, combined with Javadoc reuse, ensures
> consistency while keeping the approach lightweight.
>
> Looking forward to the refinements.
>
> Regards,
> Chinchu P Shaji
>
> From: Yeser Amer <[email protected]>
> Date: Tuesday, 23 June 2026 at 8:20 PM
> To: [email protected] <[email protected]>
> Cc: Jijo Thomas K <[email protected]>
> Subject: [EXTERNAL] Re: [HEADS-UP] Annotation of user-configurable
> properties
>
> Hello,
>
> I agree with tour proposal and changes. Thank you.
>
> Yeser
>
> Il mar 23 giu 2026, 13:35 Gabriele Cardosi <[email protected]> ha
> scritto:
>
> > Hi all,
> > thanks for feedback and kind words.
> >
> > The PRs already have the required approvals, so I'll wait until tomorrow
> > morning (CET time) to merge.
> > There will be a follow-up to further refine/improve the discovered ones,
> > but for the moment being I would prefer to close this first step to avoid
> > keeping it open too long.
> >
> > Thanks again
> >
> > Cheers
> >
> > Gabriele
> >
> >
> > On Tue, Jun 23, 2026 at 2:05 PM Alex Porcelli <[email protected]> wrote:
> >
> > > Good job Gabriele; great improvement
> > >
> > >
> > > Regards,
> > > _____________
> > > Alex Porcelli
> > >
> https://urldefense.proofpoint.com/v2/url?u=http-3A__porcelli.me&d=DwIFaQ&c=BSDicqBQBDjDI9RkVyTcHQ&r=Q6vLAuW9efZUIpzkybJzdGQcnEzNk9UeYj-IzYzMKJA&m=oDTL1C1WGe_0PNd7kQ9vo3L3h8ZmfoCqBNN62VituBusISgmmEy6p41wZ_U1RiqT&s=cFxU59z72REBUJvNP5ZHiYLwHDWubOEY4DD28k9dQAM&e=
> > >
> > > On Tue, Jun 23, 2026 at 5:21 AM Gabriele Cardosi <
> > > [email protected]>
> > > wrote:
> > >
> > > > Hi Jijo,
> > > > I've incorporated your suggestions in the Drools PR.
> > > > <
> https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_apache_incubator-2Dkie-2Ddrools_pull_6766&d=DwIFaQ&c=BSDicqBQBDjDI9RkVyTcHQ&r=Q6vLAuW9efZUIpzkybJzdGQcnEzNk9UeYj-IzYzMKJA&m=oDTL1C1WGe_0PNd7kQ9vo3L3h8ZmfoCqBNN62VituBusISgmmEy6p41wZ_U1RiqT&s=moyuiDNN2LZ_tl-DTPvEwXTglLvm8WR4bfXYQ5bWDDQ&e=
> >
> > > >
> > > > Cheers
> > > >
> > > > Gabriele
> > > >
> > > > On Tue, Jun 23, 2026 at 10:52 AM Jijo Thomas K <
> [email protected]>
> > > > wrote:
> > > >
> > > > > Thanks @Gabriele <[email protected]> for considering my
> > > > > suggestion,
> > > > >
> > > > > Field Type : What I thought was the type of the value accepted -
> > > String,
> > > > > Long, Integer, Date etc..
> > > > >
> > > > >
> > > > > Regards,
> > > > > Jijo Thomas K
> > > > >
> > > > > *From: *Gabriele Cardosi <[email protected]>
> > > > > *Date: *Monday, 22 June 2026 at 1:19 PM
> > > > > *To: *[email protected] <[email protected]>
> > > > > *Cc: *Jijo Thomas K <[email protected]>
> > > > > *Subject: *[EXTERNAL] Re: [HEADS-UP] Annotation of
> user-configurable
> > > > > properties
> > > > >
> > > > > This Message Is From an External Sender
> > > > > This message came from outside your organization.
> > > > > Report Suspicious
> > > > > <
> > > >
> > >
> >
> https://us-phishalarm-ewt.proofpoint.com/EWT/v1/AdhS1Rd-!-XFVHHhz_o9zvLrS4iWvXzBTOUgHad1qpamn1QGBUZN6vPCrFC6rigvqsqK86gbyBdcBny8evWjiD0h9qbVetA8LHattzH7sFT4uB1clidG75LhCy3uMhc3819HxDw$
> > > > >
> > > > >
> > > > > Hi all, thanks for your feedback.
> > > > > @Jijo, about your suggestions:
> > > > >
> > > > > "Accessibility" : this is managed by the creation of a second
> > > annotation
> > > > -
> > > > > "@KieInternal"; I did not mention in this thread to keep it simple,
> > but
> > > > it
> > > > > is mentioned in the issue
> > > > > <
> > > >
> > >
> >
> https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_apache_incubator-2Dkie-2Dissues_issues_2349&d=DwMFaQ&c=BSDicqBQBDjDI9RkVyTcHQ&r=chE-O7gziZQiZi9dvpCs_NiJukBA2SJazsOwhCN4Zr4&m=54dC1Xa_YDAGwAFnvv_zsJv05wYNoIKDN8qHyVlNy2UjZSi_dQ6LdOuUCUhPZVYH&s=IZ0VHHUb4t3pqexcumGdK6-KQFH9y0GLGfbKuo1_xbU&e=
> > > > >
> > > > > itself;
> > > > > "Default value": ok
> > > > > "Field type": I would like to have a specific metadata for that
> > > > > "Short description": I'm doubtful about that. There is also a PR
> > > comment
> > > > > <
> > > >
> > >
> >
> https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_apache_incubator-2Dkie-2Ddrools_pull_6766-23pullrequestreview-2D4541647613&d=DwMFaQ&c=BSDicqBQBDjDI9RkVyTcHQ&r=chE-O7gziZQiZi9dvpCs_NiJukBA2SJazsOwhCN4Zr4&m=54dC1Xa_YDAGwAFnvv_zsJv05wYNoIKDN8qHyVlNy2UjZSi_dQ6LdOuUCUhPZVYH&s=n2Fy5dfMg5vvrj4kkhh7admLKD0o64ArU-OIQaA7lvg&e=
> > > >
> > > > on
> > > > > similar terms. My point is that "@KieProperty" ->must have<-
> javadoc
> > > with
> > > > > proper description, to be available also in generated javadocs.
> And I
> > > was
> > > > > thinking to re-use such javadoc also for documentation purposes.
> > So, a
> > > > > specific "description" metadata would be redundant, IMO.
> > > > >
> > > > > Does that make sense?
> > > > >
> > > > >
> > > > > I would provide the other ones, in the meantime, making them
> optional
> > > to
> > > > > speed-up a bit this first iteration.
> > > > >
> > > > > Thanks again!
> > > > >
> > > > > Gabriele
> > > > >
> > > > >
> > > > >
> > > > > On Fri, Jun 19, 2026 at 8:26 PM Jijo Thomas K via dev <
> > > > [email protected]>
> > > > > wrote:
> > > > >
> > > > > Thank you Gabriele, for driving this initiative. I see the strong
> > value
> > > > it
> > > > > brings in terms of clarity and consistency for both KIE developers
> > and
> > > > end
> > > > > users.
> > > > >
> > > > > From a usability perspective, I believe expanding the annotation
> with
> > > > > additional metadata could make it even more powerful and
> > > > self‑documenting.
> > > > > Alongside the allowed values you’ve already proposed, I feel the
> > > > following
> > > > > fields could also be considered:
> > > > >
> > > > >   *   Accessibility: Whether the property is intended for internal
> > use
> > > > > (KIE developers) or external use (end users).
> > > > >
> > > > >   *
> > > > > Mandatory: Indicates if the property must be provided.
> > > > >   *   Default value: Specifies the default setting, complementing
> the
> > > > > allowed values list.
> > > > >
> > > > >   *   Short description: A concise explanation of the property’s
> > > purpose,
> > > > > including field type (String, integer, Long, Boolean etc).
> > > > >
> > > > > Regards,
> > > > > Jijo Thomas K
> > > > >
> > > > > From: soniya abraham via dev <[email protected]>
> > > > > Date: Friday, 19 June 2026 at 5:03 PM
> > > > > To: [email protected] <[email protected]>
> > > > > Cc: soniya abraham <[email protected]>
> > > > > Subject: [EXTERNAL] Re: [HEADS-UP] Annotation of user-configurable
> > > > > properties
> > > > >
> > > > > Hi Gabriele,
> > > > >
> > > > >
> > > > > Thank you for bringing up this idea — great work! I believe it will
> > be
> > > > > very helpful, as it will make it much easier to identify the
> > properties
> > > > and
> > > > > their corresponding values. Having a dedicated annotation will
> > simplify
> > > > > maintenance, improve clarity, and open the door for future
> > > enhancements.
> > > > > I’m confident this will add real value moving forward.
> > > > >
> > > > > Thanks,
> > > > > Soniya
> > > > >
> > > > >
> > > > >
> > > > > From: Gabriele Cardosi <[email protected]>
> > > > > Date: Friday, 19 June 2026 at 3:48 PM
> > > > > To: [email protected] <[email protected]>
> > > > > Subject: [EXTERNAL] [HEADS-UP] Annotation of user-configurable
> > > properties
> > > > >
> > > > > Hi all,
> > > > > during documentation tasks, but also in the development phase, we
> > face
> > > > the
> > > > > issue that a lot of user-configurable properties (e.g. system
> > > properties,
> > > > > application.properties) are defined as simple String inside our
> > > codebase.
> > > > > This makes both their maintenance and their knowledge hard.
> > > > > We then thought of creating a specific annotation for them.
> > > > > With that in place, such properties will be straightforward to
> > identify
> > > > and
> > > > > track, in case of addition/modification/removal.
> > > > >
> > > > > Having a specific annotation could also allow, in future, the
> > > > > implementation of other features (e.g. define what are the allowed
> > > values
> > > > > for each property).
> > > > >
> > > > > The task is defined at the following kie-issue [1]
> > > > >
> > > > > There are already two opened PRs:
> > > > > drools [2]
> > > > > kogito-runtimes [3]
> > > > >
> > > > > to have a better idea of the final result.
> > > > >
> > > > >
> > > > > Cheers
> > > > >
> > > > > Gabriele
> > > > >
> > > > > [1]
> > > > >
> > > >
> > >
> >
> https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_apache_incubator-2Dkie-2Dissues_issues_2349&d=DwIBaQ&c=BSDicqBQBDjDI9RkVyTcHQ&r=B-xdniRg_RT5v51i5IErRyekoV6Xm0zybs6yQwll-vM&m=h2Cu0VKW0RUxKNknaPvky6B73yq7z64P5OU_2NLFkn9c8fKxc53Co_gI5_nBN8Oj&s=4lBmO36qxpNHrRSkZun7nZfI2GRThNncNUA0OzWe0wY&e=
> > > > > [2]
> > > > >
> > > >
> > >
> >
> https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_apache_incubator-2Dkie-2Ddrools_pull_6766&d=DwIBaQ&c=BSDicqBQBDjDI9RkVyTcHQ&r=B-xdniRg_RT5v51i5IErRyekoV6Xm0zybs6yQwll-vM&m=h2Cu0VKW0RUxKNknaPvky6B73yq7z64P5OU_2NLFkn9c8fKxc53Co_gI5_nBN8Oj&s=B_A5UHBOUoYEjkXZVIu5yCTyZeLt_Rx5qV_nEqLt--w&e=
> > > > > [3]
> > > > >
> > > >
> > >
> >
> https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_apache_incubator-2Dkie-2Dkogito-2Druntimes_pull_4321&d=DwIBaQ&c=BSDicqBQBDjDI9RkVyTcHQ&r=B-xdniRg_RT5v51i5IErRyekoV6Xm0zybs6yQwll-vM&m=h2Cu0VKW0RUxKNknaPvky6B73yq7z64P5OU_2NLFkn9c8fKxc53Co_gI5_nBN8Oj&s=szSy-t82AQpQO_1Oaxfrkimx0VaqOT7W0_x0xYyUwYM&e=
> > > > >
> > > > >
> > > >
> > >
> >
>

Reply via email to