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

NPE during database upgrade #7512

Closed
PhantomYdn opened this issue Jun 26, 2017 · 7 comments
Closed

NPE during database upgrade #7512

PhantomYdn opened this issue Jun 26, 2017 · 7 comments

Comments

@PhantomYdn
Copy link
Contributor

PhantomYdn commented Jun 26, 2017

OrientDB Version: 2.2.20

Java Version: 7

OS: linux

We had db which run on 2.1.19. But now we tried to run it on instance of Orienteer which use 2.2.20. So OrientDB tried to convert DB and failed with the following exception:

Caused by: java.lang.NullPointerException
at java.util.Locale.<init>(Locale.java:573)
at java.util.Locale.<init>(Locale.java:602)
at com.orientechnologies.orient.core.config.OStorageConfiguration.getLocaleInstance(OStorageConfiguration.java:242)
at com.orientechnologies.orient.core.storage.impl.local.paginated.wal.ODiskWriteAheadLog$FilenameFilter.<init>(ODiskWriteAheadLog.java:144)
at com.orientechnologies.orient.core.storage.impl.local.paginated.wal.ODiskWriteAheadLog.<init>(ODiskWriteAheadLog.java:215)
at com.orientechnologies.orient.core.storage.impl.local.paginated.wal.ODiskWriteAheadLog.<init>(ODiskWriteAheadLog.java:154)
at com.orientechnologies.orient.core.storage.impl.local.paginated.OLocalPaginatedStorage.initWalAndDiskCache(OLocalPaginatedStorage.java:484)
at com.orientechnologies.orient.core.storage.impl.local.OAbstractPaginatedStorage.open(OAbstractPaginatedStorage.java:255)

What can be done to prevent this failing?

@PhantomYdn
Copy link
Contributor Author

Reason is simple. On 2.1.19 country was not required. Just checked in old version of Orienteer - it's null. So during upgrade it leads to NPE because of this code of Locale:

public Locale(String language, String country, String variant) {
        if (language== null || country == null || variant == null) {
            throw new NullPointerException();
        }
        baseLocale = BaseLocale.getInstance(convertOldISOCodes(language), "", country, variant);
        localeExtensions = getCompatibilityExtensions(language, "", country, variant);
    }

@andrii0lomakin
Copy link
Member

Guys this issue is already in progress and fixed by storage team I am assigning it back to us

andrii0lomakin added a commit that referenced this issue Jun 27, 2017
@andrii0lomakin
Copy link
Member

Fixed, @PhantomYdn could you try now?

@andrii0lomakin
Copy link
Member

Hi @PhantomYdn did you have a chance to look on it?

@andrii0lomakin
Copy link
Member

@PhantomYdn is it possible to provide some ETA when you will be able to check issue?

@andrii0lomakin
Copy link
Member

Hi @PhantomYdn could you provide an update on this issue?

@santo-it santo-it modified the milestones: 2.2.23, 2.2.x (next hotfix) Jul 6, 2017
@lvca
Copy link
Member

lvca commented Jul 12, 2017

Closing the issue because of no response. In case, please reopen it.

@lvca lvca closed this as completed Jul 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

6 participants