You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "Amemeda (via GitHub)" <gi...@apache.org> on 2026/08/06 17:13:52 UTC

[PR] Upgrade Datatables to v2.1.0 [accumulo]

Amemeda opened a new pull request, #6497:
URL: https://github.com/apache/accumulo/pull/6497

   Upgrading DataTables v1.13.11 to v 2.1.0
   
   _______________________________________________________________________________________________________________________
   Some Upgrade References: 
   https://cdn.datatables.net/2.1.0/
   https://datatables.net/blog/2024/datatables-2#Upgrading
   https://datatables.net/download/upgrade/
   
   This pr closes #4972 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade DataTables to v3.0.1 [accumulo]

Posted by "DomGarguilo (via GitHub)" <gi...@apache.org>.
DomGarguilo merged PR #6497:
URL: https://github.com/apache/accumulo/pull/6497


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade DataTables to v3.0.1 [accumulo]

Posted by "Amemeda (via GitHub)" <gi...@apache.org>.
Amemeda commented on PR #6497:
URL: https://github.com/apache/accumulo/pull/6497#issuecomment-5271316542

   ```
    Copyright (c) 2008-2026 SpryMedia Ltd
       Licensed under the MIT license (see above)
   ```
   
   All license info has been updated to 2026, however, the DataTables website has the copyright year starting at 2007 while our code is at 2008 for DataTables, Buttons, and ColReorder.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade DataTables to v2.1.0 [accumulo]

Posted by "Amemeda (via GitHub)" <gi...@apache.org>.
Amemeda commented on PR #6497:
URL: https://github.com/apache/accumulo/pull/6497#issuecomment-5208182582

   these are probably not actually in the project then 
   
   <img width="1379" height="227" alt="image" src="https://github.com/user-attachments/assets/a4701835-b3bd-49df-b7f6-ab8d9a75a6d8" />
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade DataTables to v2.1.0 [accumulo]

Posted by "DomGarguilo (via GitHub)" <gi...@apache.org>.
DomGarguilo commented on PR #6497:
URL: https://github.com/apache/accumulo/pull/6497#issuecomment-5208131763

   > _`jquery.datatables.min.js` contents needs to be updated but file is auto generated, how will this get updated?_
   
   I didn't think we had any minified (.min) files in the project (on purpose). Do you see that somewhere?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade DataTables to v3.0.1 [accumulo]

Posted by "DomGarguilo (via GitHub)" <gi...@apache.org>.
DomGarguilo commented on PR #6497:
URL: https://github.com/apache/accumulo/pull/6497#issuecomment-5269264709

   Spun up an instance on this branch and things are looking good. Only two small issues that I could find. Fixed those in https://github.com/Amemeda/accumulo/pull/1. Once those things are incorporated here i think this is good to go


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade DataTables to v3.0.1 [accumulo]

Posted by "DomGarguilo (via GitHub)" <gi...@apache.org>.
DomGarguilo commented on PR #6497:
URL: https://github.com/apache/accumulo/pull/6497#issuecomment-5284877505

   > All license info has been updated to 2026, however, the DataTables website has the copyright year starting at 2007 while our code is at 2008 for DataTables, Buttons, and ColReorder.
   
   Our license files need to match whats listed in the distribution.  The distributed files link to https://datatables.net/license/mit which has:
   ```
   Copyright (C) 2008-2026, SpryMedia Ltd.
   ```
   so our license should match exactly that.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade DataTables to v3.0.1 [accumulo]

Posted by "dlmarion (via GitHub)" <gi...@apache.org>.
dlmarion commented on PR #6497:
URL: https://github.com/apache/accumulo/pull/6497#issuecomment-5219328673

   I'm assuming that changes are coming to our monitor-related files too? 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade DataTables to v2.1.0 [accumulo]

Posted by "DomGarguilo (via GitHub)" <gi...@apache.org>.
DomGarguilo commented on PR #6497:
URL: https://github.com/apache/accumulo/pull/6497#issuecomment-5208196316

   We also added the 
   
   > these are probably not actually in the project then
   > 
   > <img alt="image" width="1379" height="227" src="https://private-user-images.githubusercontent.com/48224036/632553998-a4701835-b3bd-49df-b7f6-ab8d9a75a6d8.png?jwt=eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3ODYwNDEwMTQsIm5iZiI6MTc4NjA0MDcxNCwicGF0aCI6Ii80ODIyNDAzNi82MzI1NTM5OTgtYTQ3MDE4MzUtYjNiZC00OWRmLWI3ZjYtYWI4ZDlhNzVhNmQ4LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNjA4MDYlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjYwODA2VDE4MjUxNFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTAwNjVkNmZkYWEzM2NjZTk5NzhiZmVmYmI3MjExNDEyMWQzYTY5NjEzODJlYWMzYzFkYjYwNDQyNDFkODM3YzQmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JnJlc3BvbnNlLWNvbnRlbnQtdHlwZT1pbWFnZSUyRnBuZyJ9.d9aj86hKZXVk3-pSJcGk7GHz4XiW4ax-xsLxlLAqQzo">
   
   No, looks like those are from Hadoop. So no work to do on our end :+1: 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org