You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "DadiBit (via GitHub)" <gi...@apache.org> on 2025/08/10 09:55:01 UTC

[PR] Improve `Schema` type safety [arrow-js]

DadiBit opened a new pull request, #233:
URL: https://github.com/apache/arrow-js/pull/233

   ## What's Changed
   
   No breaking changes. Just a dead-simple helper to improve type safety a bit.
   
   "Full" type-safety like zod or drizzle (maybe, I have never used the latter) could happen, but some major refactoring would be required to work with objects:
    - drop name from field class at lower level
    - underlying data would be stored as an object of (Anonymous)`Field`s
    - access fields by key name, rather then index


-- 
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: github-unsubscribe@arrow.apache.org

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


Re: [PR] Improve `Schema` type safety [arrow-js]

Posted by "DadiBit (via GitHub)" <gi...@apache.org>.
DadiBit commented on PR #233:
URL: https://github.com/apache/arrow-js/pull/233#issuecomment-3172927209

   > Could you remove pnpm related changes from this PR? It's not required for this proposal, right?
   
   Done.
   
   > If you're interested in our package manager, let's work on it as a separated PR. See also: #189
   
   Cool, thanks! I'll be joining to convince everyone that pnpm is the best package manager :smile: 
   
   > Could you open an issue for this proposal? This is a sub task of #232 , right?
   
   It is a PR for #232, correct, I will open an issue with proper context and examples as soon as I can.
   
   


-- 
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: github-unsubscribe@arrow.apache.org

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


Re: [PR] Improve `Schema` type safety [arrow-js]

Posted by "kou (via GitHub)" <gi...@apache.org>.
kou commented on PR #233:
URL: https://github.com/apache/arrow-js/pull/233#issuecomment-3172882378

   Could you remove pnpm related changes from this PR?
   It's not required for this proposal, right?
   
   If you're interested in our package manager, let's work on it as a separated PR. See also: https://github.com/apache/arrow-js/issues/189
   
   Could you open an issue for this proposal? This is a sub task of https://github.com/apache/arrow-js/discussions/232 , right?


-- 
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: github-unsubscribe@arrow.apache.org

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