Hi Ryan
That's a good point, and definitely, it should be clearly stated in the report.
I will do a pass to update.
Thanks !
Regards
JB
On Thu, Feb 6, 2025 at 6:35 PM rdb...@gmail.com wrote:
>
> I looked through the report and left a few comments. The report says that
> this is the first repor
+1 (binding)
NB:
- I created https://github.com/apache/polaris/issues/960 to not
exclude md files from Rat scan
- I created https://github.com/apache/polaris/issues/961 to cleanup the NOTICE
Thanks !
Regards
JB
On Thu, Feb 6, 2025 at 5:54 PM Jean-Baptiste Onofré wrote:
>
> Hi folks,
>
> We did
+1
It’s great to see the progress!
I did find a couple of minor things:
- RAT checks exclude all **.md files, which I don’t think is correct.
Luckily, commenting out that line and running RAT shows that all of the
markdown files have good license headers. Because the markdown has header
Thanks all for the very informative discussion on this topic in the
community sync meeting today!
Let's have a dedicated discussion session as suggested.
Dennis: As to your doc, would you mind isolating caching concerns into a
separate section so that we could think about the main Persistence
API
I looked through the report and left a few comments. The report says that
this is the first report, but there is one from September/October last
year:
https://cwiki.apache.org/confluence/display/INCUBATOR/September2024#polaris
My main comment is that I'd like to see these reports focus on communit
Hi folks,
We did new fixes on NOTICE and LICENSE fix in the source distribution.
This is the vote for Apache Polaris 0.9.0-incubating rc5.
* This corresponds to the tag: apache-polaris-0.9.0-incubating-rc5
* https://github.com/apache/polaris/tree/apache-polaris-0.9.0-incubating-rc5
*
https://gi
