embulk-filter-column icon indicating copy to clipboard operation
embulk-filter-column copied to clipboard

The Plugin can't build because JsonPathCompiler is removed from jcenter

Open legiangthanh opened this issue 2 years ago • 7 comments

Hi fork,

Recently, we can't build this plugin because JsonPathCompiler is removed from jcenter. Currently, We work around this by compiling and publishing to a local repository.

Do you have a plan to republish this library or use another library?

Thanks.

legiangthanh avatar Jun 30 '23 01:06 legiangthanh

Hello, @legiangthanh. I'm contacting @civitaspo

You can find the source code here by the way. https://github.com/medjed/JsonPathCompiler

Japanese: X(Twitter)

hiroyuki-sato avatar Jun 30 '23 02:06 hiroyuki-sato

His answer.

I tried to do it a little while ago, but it was too old and full of errors, so I gave up... I don't feel like I can do it in my spare time, so I've been doing it for a long time... It's better to think in the direction of removing dependence on this Maybe. Japanese: X(Twitter)

So, we need to consider removing those dependencies.

https://twitter.com/hiroysato/status/1676219097963728897?s=20

hiroyuki-sato avatar Jul 04 '23 07:07 hiroyuki-sato

Hi @hiroyuki-sato, thank you for updating.

legiangthanh avatar Jul 05 '23 03:07 legiangthanh

Is there anyreason why this plugin couldn't use https://github.com/json-path/JsonPath instead? (vs JsonPathCompiler) ?

kakoni avatar Aug 21 '23 09:08 kakoni

@kakoni The original developer doesn't maintain much this plugin. So, We need to investigate the current implementation.

hiroyuki-sato avatar Aug 21 '23 11:08 hiroyuki-sato

@hiroyuki-sato I guess the main question is (in regards to this plugin), is https://github.com/medjed/JsonPathCompiler still being maintained? If not, perhaps some of the functionality (getParentPath, isProbablyJsonPath..) could be moved into JsonPathUtiland use Jayway JsonPath directly.

kakoni avatar Aug 22 '23 07:08 kakoni

@kakoni Some members chatted about this issue. It seems better to use Jayway JsonPath directly as a basic policy. But Nobody knows how to do it yet. That's why I wrote, "We need to investigate the current implementation".

hiroyuki-sato avatar Aug 22 '23 08:08 hiroyuki-sato