BUG: drop_duplicates drops name(s).

[seth-p]

Closes #10115.

[shoyer]

This check is for dealing with the API differences between Index and MultiIndex, yes? I think it would be cleaner to override the method on MultiIndex.

[seth-p]

Yes. OK, I can do that.

Alternatively, it would be great if the constructors of all Index subclasses (other than MultiIndex) accepted a names list of length one and used it to set name. That way one could always just set names without worrying about it.

[shoyer]

I would even be happy if you just explicitly check isinstance(self, MultiIndex).

I agree that your alternative solution would be preferable. That's part of a much bigger issue: #3268

[jreback]

you can dispense with this with a

return self._shallow_copy(result), should handle all meta-data

e.g. this will propogate things like freq/tz, etc as well.

[jreback]

though looking at this now, I think MultiIndex._shallow_copy is wrong

let me see if I can fix this

[jreback]

seth-p#1

[jreback]

@seth-p can you rebase and update?

[seth-p]

Afraid I do not have time to work on this. Could someone else handle it?

[sinhrks]

@seth-p OK, will do.