You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@opendal.apache.org by "fenprace (via GitHub)" <gi...@apache.org> on 2024/04/19 09:17:41 UTC

[I] `Operator::lister` is not exported for Node.js binding [opendal]

fenprace opened a new issue, #4504:
URL: https://github.com/apache/opendal/issues/4504

   According to [`bindings/nodejs/src/lib.rs`](https://github.com/apache/opendal/blob/main/bindings/nodejs/src/lib.rs) it seems `Operator::lister` is not exposed by the Node.js binding.
   
   Is there any particular reason of not doing that? Since [`Lister`](https://opendal.apache.org/docs/nodejs/classes/Lister.html) is already exported. Or is it something we will have in the future?
   
   `list_with` and `lister_with` are not exported as well.


-- 
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: commits-unsubscribe@opendal.apache.org.apache.org

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

Re: [I] `Operator::lister` is not exported for Node.js binding [opendal]

Posted by "Xuanwo (via GitHub)" <gi...@apache.org>.

Xuanwo commented on issue #4504:
URL: https://github.com/apache/opendal/issues/4504#issuecomment-2338364461

   > Well, I will check the codebase and give it a try.
   
   Thank you!
   
   > But I can't make any promises at the moment.
   
   That's how open source works. Thanks for you time anyway.
   
   > I'm new to this and have never done anything serious with rust.
   
   Please let me know if you have met blocks.


-- 
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: commits-unsubscribe@opendal.apache.org

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

Re: [I] `Operator::lister` is not exported for Node.js binding [opendal]

Posted by "fenprace (via GitHub)" <gi...@apache.org>.

fenprace commented on issue #4504:
URL: https://github.com/apache/opendal/issues/4504#issuecomment-2066251999

   > The original reason is related to #1533. But I think it's fine to expose `lister` first.
   
   Awesome! Will like to see this!
   
   BTW, is it similar to the reason why `Entry::metadata` is not exposed?
   
   Though the [documentation](https://opendal.apache.org/docs/nodejs/classes/Operator.html#stat) says:
   > Use stat if you:
   >
   > - Want detect the outside changes of path.
   > - Don’t want to read from cached metadata.
   > 
   > You may want to use `metadata` if you are working with entries returned by `Lister`. It’s highly possible that metadata you want has already been cached.
   
   Since `Entry::metadata` is not exposed, I guess there's currently no way to access cached metadata?
   


-- 
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: commits-unsubscribe@opendal.apache.org

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

Re: [I] `Operator::lister` is not exported for Node.js binding [opendal]

Posted by "suyanhanx (via GitHub)" <gi...@apache.org>.

suyanhanx commented on issue #4504:
URL: https://github.com/apache/opendal/issues/4504#issuecomment-2066599079

   Thank you for your interest in our nodejs binding! I will expose it later.


-- 
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: commits-unsubscribe@opendal.apache.org

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

Re: [I] `Operator::lister` is not exported for Node.js binding [opendal]

Posted by "Xuanwo (via GitHub)" <gi...@apache.org>.

Xuanwo commented on issue #4504:
URL: https://github.com/apache/opendal/issues/4504#issuecomment-2066345471

   > BTW, is it similar to the reason why `Entry::metadata` is not exposed?
   
   I suspect this feature doesn't exist because no one takes this task. Would you like to take a look?


-- 
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: commits-unsubscribe@opendal.apache.org

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

Re: [I] `Operator::lister` is not exported for Node.js binding [opendal]

Posted by "Xuanwo (via GitHub)" <gi...@apache.org>.

Xuanwo commented on issue #4504:
URL: https://github.com/apache/opendal/issues/4504#issuecomment-2338215505

   cc @fenprace, are you interested in implementing this?


-- 
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: commits-unsubscribe@opendal.apache.org

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

Re: [I] `Operator::lister` is not exported for Node.js binding [opendal]

Posted by "Xuanwo (via GitHub)" <gi...@apache.org>.

Xuanwo commented on issue #4504:
URL: https://github.com/apache/opendal/issues/4504#issuecomment-2066192906

   The original reason is related to https://github.com/apache/opendal/issues/1533. But I think it's fine to expose `lister` first.


-- 
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: commits-unsubscribe@opendal.apache.org

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

Re: [I] `Operator::lister` is not exported for Node.js binding [opendal]

Posted by "fenprace (via GitHub)" <gi...@apache.org>.

fenprace commented on issue #4504:
URL: https://github.com/apache/opendal/issues/4504#issuecomment-2338321991

   > cc @fenprace, are you interested in implementing this?
   
   Well, I will check the codebase and give it a try. But I can't make any promises at the moment.
   I'm new to this and have never done anything serious with rust.


-- 
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: commits-unsubscribe@opendal.apache.org

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

Re: [I] `Operator::lister` is not exported for Node.js binding [opendal]

Posted by "suyanhanx (via GitHub)" <gi...@apache.org>.

suyanhanx closed issue #4504: `Operator::lister` is not exported for Node.js binding
URL: https://github.com/apache/opendal/issues/4504


-- 
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: commits-unsubscribe@opendal.apache.org

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

Re: [I] `Operator::lister` is not exported for Node.js binding [opendal]

Posted by "suyanhanx (via GitHub)" <gi...@apache.org>.

suyanhanx commented on issue #4504:
URL: https://github.com/apache/opendal/issues/4504#issuecomment-5644030446

   closed by https://github.com/apache/opendal/pull/6769


-- 
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: commits-unsubscribe@opendal.apache.org

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