Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Maintenance #76

Open
jplatte opened this issue Apr 19, 2021 · 3 comments
Open

Maintenance #76

jplatte opened this issue Apr 19, 2021 · 3 comments

Comments

@jplatte
Copy link

jplatte commented Apr 19, 2021

It looks like this crate is hardly maintained. Many of the open issues, most of them many months old, haven't received any comments, and four PRs (the latest one from Nov 2020) are also sitting there without any feedback.

@akubera Do you intend to more actively maintain this crate soon? If not, would you be open to

  • add collaborators with write access to the repo or transfer ownership (possibly to an organization) and
  • add one or more additional owners to the crates.io package?

Since this crate is a dependency (though optional) of both diesel and SQLx, it probably won't be that hard to find people willing to help with maintenance.

@mreschke
Copy link

Agree. Need to let someone else help you with this repo so we can have a stable BigDecimal in rust.

@akubera
Copy link
Owner

akubera commented Jun 26, 2021

I'm sorry I haven't been paying attention to this project. The efficiency is quite terrible and last December I had actually started reimplementing everything using Vec's directly, instead of the relatively opaque BigInts, and then slowed down on working on that all together :-(.

I figured the reimplementation (which kind of has to happen, if we want good performance) would be the place to implement the useful things like context which would answer many of the issues people have, but obviously it's taking too long.

I'll go back to focusing on maintaining the version as-is, and I'll check into GitHub more often.

Again, sorry if this has caused any inconvenience.

Also, I'd like to everyone interested to discuss implementation details on the project gitter. Though, since nobody is using it, I wonder if there's a more popular chat solution that rustaceans would prefer?

@mreschke
Copy link

I would say BigDecimal is a popular and valuable library in the Rust ecosystem. I would be proud of that and see it as worth maintaining...already have good traction, keep it going. Deciding to re-write a project is a tough decision I know, and it takes a lot of time and effort. I do suggest chipping away at a new version if performance is really that bad, but in the mean time, put a bit of effort into the old code. I would suggest going through all these issues and fixing things up to make a good stable 1.0 of this older slower code. It's also nice that people have already fixed your code with PR's waiting for you to merge. Once 1.0 is stable I would casually work on a "next" branch for 2.0 re-write / performance booster. 1.0 could be feature frozen forever, although real bugs should always be fixed, even in frozen releases, they are important to give a bit of energy too.

Not sure about gitter or best place, but for me personally, I like all things to be on one place, right here on github. Easier for other people too. Github added a new "discussions" feature you can enable in the repo settings. Good for separating actual issues vs discussions like these.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants